-
Notifications
You must be signed in to change notification settings - Fork 92
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dev updates 2024-12-03 #550
Merged
Merged
Conversation
This file contains 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
pgvishnuram
approved these changes
Dec 4, 2024
danielhoherd
added a commit
that referenced
this pull request
Dec 4, 2024
* Update to latest kind versions * Rebuild python test dependencies * Use new circleci machine image version * Bump pre-commit hook versions, remove black hook * Expand ruff configs and apply ruff-format * Add PR description GHA workflow that we recently added to astronomer/astronomer * Setup venv dir using uv if possible (we already do this in astronomer/astronomer)
danielhoherd
added a commit
that referenced
this pull request
Dec 4, 2024
* Update to latest kind versions * Rebuild python test dependencies * Use new circleci machine image version * Bump pre-commit hook versions, remove black hook * Expand ruff configs and apply ruff-format * Add PR description GHA workflow that we recently added to astronomer/astronomer * Setup venv dir using uv if possible (we already do this in astronomer/astronomer)
danielhoherd
added a commit
that referenced
this pull request
Dec 4, 2024
* Update to latest kind versions * Rebuild python test dependencies * Use new circleci machine image version * Bump pre-commit hook versions, remove black hook * Expand ruff configs and apply ruff-format * Add PR description GHA workflow that we recently added to astronomer/astronomer * Setup venv dir using uv if possible (we already do this in astronomer/astronomer)
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
When I was starting on the git-sync volume ticket I found that we were not using ruff-format here yet, and various versions were pretty old here, so I made this branch to fix that before starting on git-sync volume work.
kind
versionsblack
hookruff
configs and applyruff-format
uv
if possible (we already do this in astronomer/astronomer)Related Issues
N/A, ktlo work.
Testing
As long as tests are passing that is enough, no manual tests or extended testing is required.
Merging
This should be merged to all supported branches.