Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix delivery_whitelist regex #5684

Closed
wants to merge 2 commits into from
Closed

Fix delivery_whitelist regex #5684

wants to merge 2 commits into from

Conversation

gonzalovilaseca
Copy link
Contributor

No description provided.

@xabbuh
Copy link
Member

xabbuh commented Sep 9, 2015

Can you please update the YAML config too?

@wouterj
Copy link
Member

wouterj commented Sep 9, 2015

👍

1 similar comment
@xabbuh
Copy link
Member

xabbuh commented Oct 1, 2015

👍

@xabbuh
Copy link
Member

xabbuh commented Oct 1, 2015

Thank you @gonzalovilaseca.

xabbuh added a commit that referenced this pull request Oct 1, 2015
This PR was submitted for the master branch but it was merged into the 2.3 branch instead (closes #5684).

Discussion
----------

Fix delivery_whitelist regex

Commits
-------

56d1768 Fix delivery_whitelist regex
@xabbuh xabbuh closed this Oct 1, 2015
xabbuh added a commit that referenced this pull request Oct 3, 2015
This PR was merged into the 2.3 branch.

Discussion
----------

fix YAML syntax highlighting

| Q             | A
| ------------- | ---
| Doc fix?      | yes
| New docs?     | no
| Applies to    | all
| Fixed tickets | #5684

In YAML, double quoted strings can contain escape sequences. Thus, we would have to escape the backslash if we wanted to use them instead of single quotes (see the Platform.sh build to spot the difference now).

Commits
-------

818d734 fix YAML syntax highlighting
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants