Skip to content

Fix verify script. - #8798

Merged
gianm merged 1 commit into
apache:masterfrom
gianm:fix-verify-script
Oct 31, 2019
Merged

gianm merged 1 commit into
apache:masterfrom
gianm:fix-verify-script

Conversation

@gianm

@gianm gianm commented Oct 31, 2019

Copy link
Copy Markdown
Contributor

Accidentally missed some quote escaping in #8794.

Accidentally missed some quote escaping in apache#8794.
@gianm gianm added the Bug label Oct 31, 2019
@gianm gianm added this to the 0.17.0 milestone Oct 31, 2019

@clintropolis clintropolis left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍 my bad for missing this in review of last PR

@gianm

gianm commented Oct 31, 2019

Copy link
Copy Markdown
Contributor Author

I think none of the tests will check this anyway; merging. I tested it locally.

@gianm
gianm merged commit e70b71c into apache:master Oct 31, 2019
jon-wei pushed a commit to jon-wei/druid that referenced this pull request Nov 28, 2019
Accidentally missed some quote escaping in apache#8794.
jon-wei added a commit that referenced this pull request Nov 28, 2019
Accidentally missed some quote escaping in #8794.
@gianm
gianm deleted the fix-verify-script branch September 23, 2022 19:24
riovic918data pushed a commit to riovic918data/druid that referenced this pull request Jun 12, 2026
Accidentally missed some quote escaping in apache#8794.
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