Skip to content

CodeQL docs: Fix ordering in side navigation bar for Query help #19270

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 1 commit into from
Apr 10, 2025

Conversation

felicitymay
Copy link
Contributor

@felicitymay felicitymay commented Apr 10, 2025

This list would currently be incorrectly published as shown in the screenshot below. This PR corrects the order to alphabetic. So "GitHub Actions" should appear after C/C++ and C#.

Screenshot showing 'GitHub Actions' appearing at the top of the list instead of correct alphabetic order.

@Copilot Copilot AI review requested due to automatic review settings April 10, 2025 12:12
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot wasn't able to review any files in this pull request.

Files not reviewed (1)
  • docs/codeql/query-help/index.rst: Language not supported

Copy link
Contributor

@coadaflorin coadaflorin left a comment

Choose a reason for hiding this comment

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

The screenshot LGTM!
But how does it make it to the top of the list by changing this order here? Here it doesn't seem to be first in the list.

@felicitymay
Copy link
Contributor Author

Apologies for the poor PR description. I've updated it.

The screenshot is "before". This PR should move "GitHub Actions" so that it's third in the list.

@felicitymay felicitymay merged commit c2baf9a into main Apr 10, 2025
7 checks passed
@felicitymay felicitymay deleted the felicitymay-patch-1 branch April 10, 2025 14:10
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.

3 participants