Skip to content

Aeisenberg/merge rc3.7 into main #10496

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 36 commits into from
Sep 21, 2022

Conversation

aeisenberg
Copy link
Contributor

See #10478. rc/3.7 is a protected branch and I don't want to push up the merge commit for main.

MathiasVP and others added 30 commits September 6, 2022 14:33
…stack-address-2

C++: Speedup 'cpp/using-expired-stack-address' by avoiding a large ne…
Release preparation for version 2.10.5
This moves the following three PRs to the 3.7 branch:

- github#10182
- github#10146
- github#10105
Add a better example for `registries`.
- Add examples to filter on `@id`
- Add examples that include regexes
- Add examples that include both lists and single elements
- Add some `import` examples
- Remove mention of `eval` since it is not a user-facing instruction
…workspace

Add information about the `registries` block in `codeql-workspace.yml`
Co-authored-by: Felicity Chapman <felicitymay@github.com>
Co-authored-by: Felicity Chapman <felicitymay@github.com>
Co-authored-by: Felicity Chapman <felicitymay@github.com>
Co-authored-by: Andrew Eisenberg <aeisenberg@github.com>
@@ -7,7 +7,7 @@ For example, to check for CodeQL databases using the `GitHub CLI <https://cli.gi

This command returns information about any CodeQL databases that are available for a repository, including the language the database represents, and when the database was last updated. If no CodeQL databases are available, the response is empty.

When you have confirmed that a CodeQL database exists for the language you are interested in, you can download it using the following command::
When you have confirmed that a CodeQL database exists for the language you are interested in, you can download it using the following command::

gh api /repos/<owner>/<repo>/code-scanning/codeql/databases/<language> -H 'Accept: application/zip' > path/to/local/database.zip
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jf205 Is the link below correct?

https://docs.github.com/en/rest/code-scanning#get-a-codeql-database-for-a-repository does not show up in the existing docs. Is this something yet to be released?

Copy link
Contributor

Choose a reason for hiding this comment

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

That link should be correct. It did appear on the docs page a few days ago. I've asked the docs team for assistance.
Since the problem is on docs.github.com we can merge this still merge this.

@aeisenberg aeisenberg force-pushed the aeisenberg/merge-rc3.7-into-main branch from 68e7ff7 to 61b5a45 Compare September 20, 2022 15:46
hvitved
hvitved previously approved these changes Sep 20, 2022
Copy link
Contributor

@hvitved hvitved left a comment

Choose a reason for hiding this comment

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

C# 👍

@aeisenberg
Copy link
Contributor Author

Thanks for the review. I'll wait to hear back from James before merging.

Copy link
Contributor

@jf205 jf205 left a comment

Choose a reason for hiding this comment

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

Thanks @hvitved and @aeisenberg ❤️

@aeisenberg aeisenberg merged commit 99e8cb7 into github:main Sep 21, 2022
@aeisenberg aeisenberg deleted the aeisenberg/merge-rc3.7-into-main branch September 21, 2022 15:09
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.

6 participants