Skip to content

Add repository filter by full name#1705

Merged
koesie10 merged 1 commit intomainfrom
koesie10/filter-repositories-by-name
Nov 4, 2022
Merged

Add repository filter by full name#1705
koesie10 merged 1 commit intomainfrom
koesie10/filter-repositories-by-name

Conversation

@koesie10
Copy link
Copy Markdown
Member

@koesie10 koesie10 commented Nov 3, 2022

This adds a new textbox to the outcome panels that allows filtering by the repository full name (e.g. github/vscode-codeql). The filtering uses the same logic as the existing remote queries filter, i.e. by converting the input and the repository full name to lower case and checking the the latter includes the former.

Screenshot 2022-11-03 at 16 22 06

Checklist

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

This adds a new textbox to the outcome panels that allows filtering by
the repository full name (e.g. `github/vscode-codeql`). The filtering
uses the same logic as the existing remote queries filter, i.e. by
converting the input and the repository full name to lower case and
checking the the latter includes the former.
@koesie10 koesie10 added the secexp label Nov 3, 2022
@koesie10 koesie10 requested review from a team as code owners November 3, 2022 10:18
Base automatically changed from koesie10/add-metadata-to-repo-row to main November 3, 2022 10:55
@koesie10 koesie10 merged commit c0ba9c5 into main Nov 4, 2022
@koesie10 koesie10 deleted the koesie10/filter-repositories-by-name branch November 4, 2022 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants