Skip to content

feat(dev): alembic check #16483

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

Merged
merged 4 commits into from
Aug 15, 2024
Merged

Conversation

@miketheman miketheman added testing Test infrastructure and individual tests developer experience Anything that improves the experience for Warehouse devs labels Aug 14, 2024
@miketheman miketheman requested a review from a team as a code owner August 14, 2024 15:00
@miketheman
Copy link
Member Author

miketheman commented Aug 14, 2024

Note: this check should not be marked as required, since there are cases where we want to allow for a code-side removal (which would generate a migration) and merge that PR, and ship the database migration after the code no longer relies on it.

Upon review, we added the ability to skip the check if a label is applied, and thus make this a required check.

@miketheman miketheman force-pushed the miketheman/alembic-check branch 2 times, most recently from 9022510 to 2a60833 Compare August 14, 2024 20:00
@miketheman
Copy link
Member Author

miketheman commented Aug 14, 2024

@miketheman miketheman merged commit 42a6ad1 into pypi:main Aug 15, 2024
18 checks passed
@miketheman miketheman deleted the miketheman/alembic-check branch August 15, 2024 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
developer experience Anything that improves the experience for Warehouse devs testing Test infrastructure and individual tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants