Skip to content

Release Tracking #298

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 1 commit into from
May 9, 2025
Merged

Release Tracking #298

merged 1 commit into from
May 9, 2025

Conversation

openprojectci
Copy link

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.

Releases

@openproject/[email protected]

Minor Changes

Patch Changes

Copy link

github-actions bot commented May 7, 2025

❗ Pre-merge checklist

Please ensure these items are checked before merging.

🔎 Smoke test

  • All CI checks pass
  • Lookbook opens in a browser
  • Successful integration test with OpenProject as a primary consumer
    • Install the npm package locally
    • Install the primer_view_components gem locally
    • Verify no new build errors appear
    • Verify no new linting errors appear
    • Manually test critical paths
    • Manually test release-specific bugfixes and/or features work as described

🤔 Sanity test

  • All bugfixes in this release have resolved their corresponding issues
  • All new features in this release have been tested and verified as compatible with OpenProject
  • No noticeable regressions have been introduced as a result of changes in this release
  • Release notes accurately describe the changes made

🚢 Post ship reminders

  • Deploy lookbook via chat ops in the #primer-rails-ops channel. .deploy primer-lookbook to production

Please also leave any testing notes as a comment on this pull request. In particular, describing any issues encountered during your testing. This is helpful in providing historical context to maintainers.

@HDinger
Copy link
Collaborator

HDinger commented May 7, 2025

Hi @camertron

@myabc and I found the reason for the failing release CI. The Changeset team introduced a bug some days ago in this commit. Basically they switched from

git push origin --tags
to
git push origin v.0.65.0

The first one just does nothing when there is no local tag while the second creates a failure and stops the execution.
Alex creates an issue in their repo and we will wait for their response. There is already an issue, and Alex will leave a comment. We will wait a bit and see whether they react. If not, we will create the tag manually in the changset-publish script. Then we should be able to release again.

@myabc
Copy link

myabc commented May 8, 2025

@camertron @HDinger my comment basically reiterates what @HDinger stated in this PR.

@camertron
Copy link
Collaborator

camertron commented May 8, 2025

Hey @HDinger and @myabc, nice work tracking down the issue! I will say, this is not the first time the changesets tool has exhibited weird behavior 😬

@HDinger
Copy link
Collaborator

HDinger commented May 9, 2025

I also left a comment in the upstream PVC repo.

@openprojectci openprojectci force-pushed the changeset-release/main branch from 6caa975 to 31a4cdf Compare May 9, 2025 06:32
@HDinger HDinger merged commit dba79cb into main May 9, 2025
28 checks passed
@HDinger HDinger deleted the changeset-release/main branch May 9, 2025 06:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants