Skip to content

joh/appalling tapir#140

Merged
jrieken merged 4 commits intomainfrom
joh/appalling-tapir
Oct 3, 2022
Merged

joh/appalling tapir#140
jrieken merged 4 commits intomainfrom
joh/appalling-tapir

Conversation

@jrieken
Copy link
Member

@jrieken jrieken commented Oct 3, 2022

@jrieken jrieken self-assigned this Oct 3, 2022
@jrieken jrieken enabled auto-merge (squash) October 3, 2022 09:13
['draft', QualifiedValueInfo.enum(new ValueSet(true, 'true', 'false'), undefined, 'Draft pull requests')],
['in', QualifiedValueInfo.enum(new ValueSet(true, 'title', 'body', 'comments'), undefined, 'Search in the title, body, comments, or any combination of these')],
['is', QualifiedValueInfo.enum([new ValueSet(true, 'locked', 'unlocked'), new ValueSet(true, 'merged', 'unmerged'), new ValueSet(true, 'public', 'private'), new ValueSet(true, 'open', 'closed'), new ValueSet(true, 'pr', 'issue')], RepeatInfo.Repeat)],
['reason', QualifiedValueInfo.enum(new ValueSet(true, 'closed', '"not planned"'))],
Copy link
Member

Choose a reason for hiding this comment

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

Should this be completed?

assertValidateErrors('-label:', Code.NodeMissing);
assertValidateErrors('label:foo,bar');
assertValidateErrors('milestone:foo,bar', Code.SequenceNotAllowed);
assertValidateErrors('reason:closed');
Copy link
Member

Choose a reason for hiding this comment

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

Same here - completed.

Copy link
Member

@lszomoru lszomoru left a comment

Choose a reason for hiding this comment

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

🚀

@jrieken jrieken merged commit 60aad37 into main Oct 3, 2022
@jrieken jrieken deleted the joh/appalling-tapir branch October 3, 2022 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants