-
Notifications
You must be signed in to change notification settings - Fork 4.5k
airbyte-lib: Add publish action #34486
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
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Ignored Deployment
|
…ing' into flash1293/airbyte-lib-publish2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good!
## Release | ||
|
||
* In your PR: | ||
* Bump the version in `pyproject.toml` | ||
* Add a changelog entry to the table below | ||
* Once the PR is merged, go to Github and trigger the `Publish AirbyteLib Manually` workflow. This will publish the new version to PyPI. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we get CI checks for missing changelog entries and/or unbumped versions?
It would be great if we could piggy-back on existing rails, if that's possible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's not really good existing guardrails as all places kind of do their own thing (connectors, airbyte-cdk). However, I added a basic check that a changelog entry is set for the current version.
I kept this simple deliberately so we have control over the publishing process in the first messy phase. We can add some automation later on if it makes sense (like auto-publishing as soon as the version changes on merge to master).
Co-authored-by: alafanechere <[email protected]>
Co-authored-by: alafanechere <[email protected]>
Co-authored-by: alafanechere <[email protected]>
Add publish action for airbyte-lib to release on pypi