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

Update docs on the output of resolve qlpacks #7571

Merged
merged 2 commits into from Jan 24, 2022
Merged

Conversation

@aeisenberg
Copy link
Contributor

@aeisenberg aeisenberg commented Jan 11, 2022

The output has changed and there are no more upgrades
packs. There are also other changes included here.

Related to github/semmle-code#40918

The output has changed and there are no more upgrades
packs. There are also other changes included here.
@aeisenberg aeisenberg force-pushed the aeisenberg/remove-upgrades branch from 4d457ca to e4eb2c2 Jan 11, 2022
Copy link
Contributor

@felicitymay felicitymay left a comment

Many thanks for the docs update

I've made a suggestion with some minor editorial fixes, but otherwise this looks ready to merge.

Presumably this will ship with the CodeQL CLI release that's expected at the end of January/early February, but can be merged now?

@aeisenberg
Copy link
Contributor Author

@aeisenberg aeisenberg commented Jan 13, 2022

Presumably this will ship with the CodeQL CLI release that's expected at the end of January/early February, but can be merged now?

Yes. The change doesn't mention the upgrades packs. These packs exist in CLI 2.7.5 and will be removed in 2.7.6.

@aeisenberg
Copy link
Contributor Author

@aeisenberg aeisenberg commented Jan 24, 2022

v2.7.6 is released now. I will merge these docs changes so they will be released along with v2.8.0

@aeisenberg aeisenberg merged commit 497c878 into main Jan 24, 2022
15 checks passed
@aeisenberg aeisenberg deleted the aeisenberg/remove-upgrades branch Jan 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants