Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Release tools - clear cache before asking an npm registry #17328

Closed
pomek opened this issue Oct 25, 2024 · 0 comments · Fixed by ckeditor/ckeditor5-dev#1036
Closed

Release tools - clear cache before asking an npm registry #17328

pomek opened this issue Oct 25, 2024 · 0 comments · Fixed by ckeditor/ckeditor5-dev#1036
Assignees
@pomek
Labels
package:dev squad:platform Issue to be handled by the Platform team. type:bug This issue reports a buggy (incorrect) behavior.
Milestone
iteration 80 (v44...

Comments

@pomek
Copy link
Member

pomek commented Oct 25, 2024

📝 Provide detailed reproduction steps (if any)

The latest ckeditor5-dev release "failed" while a script tried to publish packages.

https://app.circleci.com/pipelines/github/ckeditor/ckeditor5-dev/1297/workflows/1f204301-e5cd-4a64-8dbf-cc1b99c75f66/jobs/2485

After restarting the build, nothing crashed, but nothing was uploaded.

https://app.circleci.com/pipelines/github/ckeditor/ckeditor5-dev/1297/workflows/1c55e053-e198-460c-8019-940678099df8

This means that the first workflow was processed successfully with the release.

To avoid false-positives, let's clear the cache before communicating with the registry.
@pomek pomek added package:dev squad:platform Issue to be handled by the Platform team. type:bug This issue reports a buggy (incorrect) behavior. labels Oct 25, 2024
@pomek pomek mentioned this issue Oct 25, 2024
Merged
@CKEditorBot CKEditorBot added the status:in-progress Set automatically when an issue lands in the "In progress" column. We are working on it. label Oct 25, 2024
@pomek pomek self-assigned this Oct 25, 2024
pomek added a commit to ckeditor/ckeditor5-dev that referenced this issue Oct 25, 2024
@pomek
Merge pull request #1036 from ckeditor/i/17328
90a1032 
Internal (release-tools): Do not use the cached values when asking the npm registry about a package. Closes ckeditor/ckeditor5#17328.

Internal: Changelog for v45.0.3.
@CKEditorBot CKEditorBot removed the status:in-progress Set automatically when an issue lands in the "In progress" column. We are working on it. label Oct 25, 2024
@CKEditorBot CKEditorBot added this to the iteration 80 milestone Oct 25, 2024
This was referenced Oct 25, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@pomek pomek

Labels
package:dev squad:platform Issue to be handled by the Platform team. type:bug This issue reports a buggy (incorrect) behavior.
Projects
None yet
Milestone
iteration 80 (v44.0.0)
Development

Successfully merging a pull request may close this issue.

Disable cache when talking with the npm registry ckeditor/ckeditor5-dev
2 participants
@pomek @CKEditorBot