-
Notifications
You must be signed in to change notification settings - Fork 1
[DPE-8030] Publishing workflow #6
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
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## 16/edge #6 +/- ##
==========================================
Coverage ? 32.21%
==========================================
Files ? 3
Lines ? 776
Branches ? 91
==========================================
Hits ? 250
Misses ? 518
Partials ? 8 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
name = "postgresql-charms-single-kernel" | ||
dynamic = ["version"] | ||
description = "Shared and reusable code for PostgreSQL-related charms" | ||
version = "0.0.1" |
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.
Ldap sync manually bumps the version.
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.
Mostly copied from postgresql-ldap-sync.
with: | ||
fetch-depth: 0 | ||
- name: "Install uv" | ||
run: sudo snap install astral-uv --classic |
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.
Using UV instead of poetry.
DPW publishing flow depends on poetry: https://github.com/canonical/data-platform-workflows/blob/main/.github/workflows/release_python_package.md |
run: | | ||
VERSION=$(uv version --short) | ||
if [ "$(git tag -l "${VERSION}")" ]; then | ||
echo "Tag ${VERSION} already exists. Please bump the project to a greater version." |
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.
Always writes error to STDERR: >&2
Copy the release workflow from postgresql-ldap-sync
Checklist