Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(scripts): Support jq version 1.6 #10785

Merged
merged 2 commits into from
Dec 30, 2024

Conversation

gibson042
Copy link
Member

#10590 (comment)

I also noticed we need jq 1.7 minimum when running scripts/npm-dist-tag.sh and my machine had jq 1.6 out of the box.

Description

Replace pick(...) with an equivalent expression that doesn't require version 1.7 (cf. jq/NEWS.md and jqlang/jq#2656 ).

Security Considerations

None.

Scaling Considerations

n/a

Documentation Considerations

Deferred.

Testing Considerations

Not covered.

Upgrade Considerations

n/a

Copy link
Member

@mujahidkay mujahidkay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the quick PR!

@gibson042 gibson042 added the automerge:squash Automatically squash merge label Dec 30, 2024
Copy link

cloudflare-workers-and-pages bot commented Dec 30, 2024

Deploying agoric-sdk with  Cloudflare Pages  Cloudflare Pages

Latest commit: cb1d554
Status: ✅  Deploy successful!
Preview URL: https://3a6da65b.agoric-sdk.pages.dev
Branch Preview URL: https://gibson-2024-12-npm-dist-tag.agoric-sdk.pages.dev

View logs

@mergify mergify bot merged commit ca25dd5 into master Dec 30, 2024
81 checks passed
@mergify mergify bot deleted the gibson-2024-12-npm-dist-tag-jq-version branch December 30, 2024 19:31
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
automerge:squash Automatically squash merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants