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.

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

Fix jq version evaluation #866

Merged
merged 4 commits into from
Mar 21, 2024
Merged

Fix jq version evaluation #866

merged 4 commits into from
Mar 21, 2024

Conversation

Jefajers
Copy link
Member

Overview/Summary

This PR closes #865 by changing the logic used to determine jq --version output.

This PR fixes/adds/changes/removes

  1. Changes Assert-AzOpsJqDependency.ps1

Breaking Changes

  1. N/A

Testing Evidence

Testing with a jq version variable set to 1.7.1 evaluates correct.

As part of this Pull Request I have

  • Checked for duplicate Pull Requests
  • Associated it with relevant issues, for tracking and closure.
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Performed testing and provided evidence.
  • Updated relevant and associated documentation.

@Jefajers Jefajers added the bug Something isn't working label Mar 20, 2024
@Jefajers Jefajers added this to the v2.6.0 milestone Mar 20, 2024
@Jefajers Jefajers requested review from a team as code owners March 20, 2024 10:46
daltondhcp
daltondhcp previously approved these changes Mar 20, 2024
@Jefajers Jefajers merged commit c37e9fd into Azure:main Mar 21, 2024
4 checks passed
@Jefajers Jefajers deleted the jqDependency branch March 21, 2024 07:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Issue evaluating jq version
2 participants