Lint entire project, not just dags dir by jeremybeard · Pull Request #1840 · astronomer/astro-cli
Navigation Menu
{{ message }}
astronomer / astro-cli Public
- Notifications You must be signed in to change notification settings
- Fork 106
Merged
Lint entire project, not just dags dir#1840
jeremybeard merged 1 commit intomainfrom
Lint entire project, not just dags dir#1840
jeremybeard merged 1 commit intomainfrom
Conversation
Copy link
Contributor
jeremybeard
commented
Apr 18, 2025
jeremybeard
commented
Description
This changes the new ruff linting feature for astro dev upgrade-test to lint the entire project, not just the dags/ directory. This will capture issues in other files such as in include/.
📋 Checklist
- Rebased from the main (or release if patching) branch (before testing)
- Ran
make testbefore taking out of draft - Ran
make lintbefore taking out of draft - Added/updated applicable tests
- Tested against Astro-API (if necessary).
- Tested against Houston-API and Astronomer (if necessary).
- Communicated to/tagged owners of respective clients potentially impacted by these changes.
- Updated any related documentation
jeremybeard
requested review from
kushalmalani and
neel-astro
as code owners
neel-astro approved these changes Apr 18, 2025
jeremybeard
deleted the
lint-whole-project
branch
neel-astro pushed a commit that referenced this pull request
Apr 24, 2025This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment