-
Notifications
You must be signed in to change notification settings - Fork 1
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
Feat: automate releases #74
Conversation
Fix lint in release-source.yml Fix lint in release-source.yml Fix lint in release-source.yml Fix version validation error in release-source.yml Fix lint Fix lint in release-source.yml Remove empty line in release-source workflow Update release_version input to be required in release-source.yml Update release-source workflow Update release-source workflow inputs Update release-source workflow to use workflow_dispatch event Update release-source workflow Fix file_release_version extraction in release-source.yml Add version increment logic Fix version comparison bug Fix version validation in release-source.yml extend version comparison logic in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml Fix version comparison bug in release-source.yml
Fix typo fix helm dep fix helm dep fix helm build fix helm build add chartmuseum token add chartmuseum Add CHARTMUSEUM_TOKEN to environment variables chore: Update Chart.yaml with release version for marvinjs fix typo fix typo fix lint
@dvvanessastoiber fix is committed |
@dvviktordelev Thank you! I just tested it with reprovisyn, but it seems to have the same issue as before. This version should have been updated as well: https://github.com/datavisyn/reprovisyn/pull/2034/files#diff-c0de79f44a4fa1dc584a8978681a811be1d71b3feccebc57308fea2bd5c5cf87R33. Please let me know if you have any questions. |
@dvvanessastoiber changes are committed, I also added a lot of debug messages in case of failure. Will need to remove them after successful run |
@dvviktordelev Thank you! The logs are very helpful. I just tried again and it looks like it just finds the name of the dependency, not the corresponding version (see https://github.com/datavisyn/reprovisyn/actions/runs/11158895196/job/31016163066):
|
@dvvanessastoiber changes are committed. please try again. |
@dvviktordelev Thank you! Unfortunately, it still has the same behavior. Could you test this particular replacement part locally? |
@dvvanessastoiber fix is committed! |
.github/workflows/release-source.yml
Outdated
PRS=$(curl -s -H "Accept: application/vnd.github+json" \ | ||
-H "Authorization: Bearer ${{ secrets.GITHUB_TOKEN }}" \ | ||
-H "X-GitHub-Api-Version: 2022-11-28" \ | ||
"https://api.github.com/repos/${{ github.repository_owner }}/${{ github.event.repository.name }}/pulls?state=closed&base=develop&sort=updated&direction=desc") |
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.
Please use the input branch (github.ref_name
) here instead of hardcoded develop. Thank you!
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.
fixed and committed
…visyn/github-workflows into feat-automate-releases-releasenotes
This is based on previous PR
it adds dynamic creation of release notes (= list of closed PRs since last release)
closes https://github.com/datavisyn/infrastructure-k8s/issues/241