Skip to content

escape metacharacters in job name#3772

Merged
vilmibm merged 1 commit intocli:trunkfrom
f1rstmehul:escape-metacharacters
Jun 2, 2021
Merged

escape metacharacters in job name#3772
vilmibm merged 1 commit intocli:trunkfrom
f1rstmehul:escape-metacharacters

Conversation

@f1rstmehul
Copy link
Contributor

@f1rstmehul f1rstmehul commented Jun 2, 2021

fixes #3736

mislav
mislav approved these changes Jun 2, 2021
View reviewed changes
Copy link
Contributor

@mislav mislav left a comment

Choose a reason for hiding this comment

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

Thanks! This looks good.

@vilmibm vilmibm merged commit 4d90005 into cli:trunk Jun 2, 2021
@f1rstmehul f1rstmehul deleted the escape-metacharacters branch June 3, 2021 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@mislav mislav mislav approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

gh run view --log throws regex parsing error when job name contains ++

3 participants

@f1rstmehul @mislav @vilmibm