Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

search jobs: fail validation for select: queries#60924

Merged
stefanhengl merged 1 commit into
sh/search-jobs/support-diff-commitfrom
sh/search-jobs/check-for-ownership-queries
Mar 8, 2024
Merged

search jobs: fail validation for select: queries#60924
stefanhengl merged 1 commit into
sh/search-jobs/support-diff-commitfrom
sh/search-jobs/check-for-ownership-queries

Conversation

@stefanhengl

@stefanhengl stefanhengl commented Mar 7, 2024

Copy link
Copy Markdown
Member

It is currently possible to create search jobs based on select: queries although we don't support them. This adds a check to fail validation.

Test plan

  • updated unit test

@cla-bot cla-bot Bot added the cla-signed label Mar 7, 2024
@stefanhengl stefanhengl requested a review from a team March 7, 2024 13:41
@stefanhengl stefanhengl marked this pull request as ready for review March 7, 2024 13:41
@stefanhengl stefanhengl merged commit ac61533 into sh/search-jobs/support-diff-commit Mar 8, 2024
@stefanhengl stefanhengl deleted the sh/search-jobs/check-for-ownership-queries branch March 8, 2024 07:59
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants