-
Notifications
You must be signed in to change notification settings - Fork 30.2k
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
deps: update icu to #50495
deps: update icu to #50495
Conversation
update ICU to what? 😄 I guess it's a bug, also it's not updating the maintaining-dependencies list |
https://github.com/nodejs/node/actions/runs/6717282091/job/18254870034
The script should probably end with an error if there is a checksum problem. |
I'd like to retry it. |
Otherwise we end up with an bad update PR when checksums don't match. Refs: nodejs#50495
Otherwise we end up with an bad update PR when checksums don't match. Refs: #50495 PR-URL: #50522 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
Otherwise we end up with an bad update PR when checksums don't match. Refs: nodejs#50495 PR-URL: nodejs#50522 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
Otherwise we end up with an bad update PR when checksums don't match. Refs: #50495 PR-URL: #50522 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
Otherwise we end up with an bad update PR when checksums don't match. Refs: #50495 PR-URL: #50522 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
Otherwise we end up with an bad update PR when checksums don't match. Refs: #50495 PR-URL: #50522 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Luigi Pinca <[email protected]>
Why didn't it match? was the release broken or transient err? |
It was #50498 (missing checksum files in the release artifacts) |
Thanks. 74.2 had a broken LICENSE file, not sure if that has been corrected. |
The problem with 74.2 is that it's still a Pre-release on GitHub: https://github.com/unicode-org/icu/releases |
FWIW I opened an issue for this last week: https://unicode-org.atlassian.net/browse/ICU-22622 |
cross linked to another issue. |
This is an automated update of icu to .