pierrejeambrun commented on PR #45696: URL: https://github.com/apache/airflow/pull/45696#issuecomment-2605160908
> Question for the audience :) > I am looking into updating the basic tests to include tests from the simple auth manager UI as part of the [tests-ui](https://github.com/apache/airflow/blob/main/.github/workflows/basic-tests.yml#L87). Some caching is done that I am not familiar with. Do I need to do the same or just I am not super familiar with this workflow but by the look of it I would say that caching the node_modules is always helpful. I think we can just follow the same logic and apply that to the `airflow/auth/managers/simple/ui/node_modules` folder. (basically duplicating the Restore and Save cache before trying to `pnpm install`) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
