Skip to content

Solorigate: Extract to separate qlpack #7431

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

Merged
merged 3 commits into from
Jan 6, 2022

Conversation

aeisenberg
Copy link
Contributor

Extracts solorigate to separate qlpacks in preparation for publishing them to the registry.

@aeisenberg aeisenberg requested a review from a team as a code owner December 16, 2021 21:43
@aeisenberg aeisenberg marked this pull request as draft December 16, 2021 21:43
@aeisenberg aeisenberg removed the request for review from a team December 16, 2021 21:44
@aeisenberg aeisenberg force-pushed the aeisenberg/solorigate-publish branch from 9b41b5c to d8ee792 Compare December 16, 2021 21:55
@github-actions github-actions bot added the JS label Dec 16, 2021
@aeisenberg aeisenberg force-pushed the aeisenberg/solorigate-publish branch from d8ee792 to c5de00b Compare December 17, 2021 00:05
Extracts solorigate to separate qlpacks in preparation for
publishing them to the registry.
@aeisenberg aeisenberg force-pushed the aeisenberg/solorigate-publish branch from c5de00b to 50ee4ab Compare December 17, 2021 00:09
@aeisenberg aeisenberg marked this pull request as ready for review December 17, 2021 00:18
@aeisenberg aeisenberg requested a review from a team as a code owner December 17, 2021 00:18
@aeisenberg aeisenberg requested review from dbartol and a team December 17, 2021 00:18
@aeisenberg
Copy link
Contributor Author

aeisenberg commented Dec 17, 2021

Solorigate queries are now available from the package registry. Run them like this:

codeql database analyze --format=sarif-latest --output=out --download <any-csharp-db> codeql/csharp-solorigate-queries

@aeisenberg
Copy link
Contributor Author

This PR also includes a simple script to re-publish and version bump.

@@ -0,0 +1,9 @@
name: codeql/solorigate-queries
Copy link
Contributor

Choose a reason for hiding this comment

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

should this mention csharp?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good point. I made the change. Unpublished previous packs and re-published under the new name.

Copy link
Contributor

Choose a reason for hiding this comment

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

Did you forget to push the change?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I pushed to the wrong remote. Should be good now.

@@ -0,0 +1,8 @@
name: codeql/solorigate-all
Copy link
Contributor

Choose a reason for hiding this comment

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

should this mention csharp?

@aeisenberg aeisenberg force-pushed the aeisenberg/solorigate-publish branch from 2d35ec4 to 7a38618 Compare January 4, 2022 16:28
Comment on lines 9 to 10
codeql/solorigate-all: "*"
codeql/solorigate-queries: "*"
Copy link
Contributor

Choose a reason for hiding this comment

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

These should be updated, right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. Done.

@aeisenberg
Copy link
Contributor Author

Thanks @hvitved for reviewing!

@aeisenberg aeisenberg merged commit 6d62227 into github:main Jan 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants