Skip to content

Comments

Lint entire project, not just dags dir#1840

Merged
jeremybeard merged 1 commit intomainfrom
lint-whole-project
Apr 18, 2025
Merged

Lint entire project, not just dags dir#1840
jeremybeard merged 1 commit intomainfrom
lint-whole-project

Conversation

@jeremybeard
Copy link
Contributor

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 test before taking out of draft
  • Ran make lint before 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 jeremybeard merged commit ad0ffeb into main Apr 18, 2025
3 of 4 checks passed
@jeremybeard jeremybeard deleted the lint-whole-project branch April 18, 2025 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants