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

Add exec permissions for installed jq #1

Merged
merged 1 commit into from
Nov 2, 2022
Merged

Add exec permissions for installed jq #1

merged 1 commit into from
Nov 2, 2022
+2 −0

Conversation

goruha
Copy link
Contributor

@goruha goruha commented Nov 2, 2022 •
edited
Loading

@goruha
Add exec permissions for installed jq

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
GPG key ID: B5690EEEBB952194
Verified
Learn about vigilant mode
2729c75
@goruha goruha requested a review from dcarbone as a code owner November 2, 2022 19:56
@dcarbone
Copy link
Owner

dcarbone commented Nov 2, 2022

Mm, good point on the test. Will add that.

@dcarbone dcarbone merged commit 91d8da7 into dcarbone:main Nov 2, 2022
@dcarbone dcarbone mentioned this pull request Nov 2, 2022
Closed
@dcarbone
Copy link
Owner

dcarbone commented Nov 2, 2022

Basic tests for this added. Thanks for PR and finding this issue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dcarbone dcarbone

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@goruha @dcarbone