You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
minor #16457 [Messenger] tidy MESSENGER_CONSUMER_NAME surrounds (PhilETaylor)
This PR was submitted for the 6.0 branch but it was merged into the 5.4 branch instead.
Discussion
----------
[Messenger] tidy MESSENGER_CONSUMER_NAME surrounds
I am an idiot.
I have a 27" Mac so the docs renders max-width - looks like this:
<img width="816" alt="Screenshot 2022-01-30 at 01 29 44" src="https://user-images.githubusercontent.com/400092/151683246-a7f45a93-7fdc-4765-96e6-df948d2f2ebc.png">
When I see
> can be set to an environment variable (like %env(MESSENGER_CONSUMER_NAME)%) set by Supervisor
due to the way the page flow of this paragraph is (see screenshot), I copy and pasted `%env(MESSENGER_CONSUMER_NAME)%)` which obviously has a confusing, and not instantly spotted closing bracket.
This PR simply removes the brackets in favour of commas, so that the next person doesnt waste 30 mins trying to work out why his env vars were not being replaced.
take it or leave it, if you dont think the PR has merit, or if you agree Im an idiot, then no worries, it can be closed :)
Commits
-------
6d80ef9 tidy MESSENGER_CONSUMER_NAME
0 commit comments