Skip to content

Update security_checker.rst #11139

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

Closed
wants to merge 2 commits into from
Closed

Conversation

shreypuranik
Copy link
Contributor

Adding a small note advising users to always use the latest version of the command, as old versions will now break existing build plans because change of composer.lock checking tool.

Adding a small note advising users to always use the latest version of the command, as old versions will now break existing build plans because change of composer.lock checking tool.
@javiereguiluz
Copy link
Member

@shreypuranik thanks for this new contribution!

While merging it in 3.4 branch, we did some rewords (details: 858b1a7). We regularly do this with most contributions (we reword them to make them more actionable, or more concise, or more "aligned" with the existing docs), so it's not specific to your contribution :)

Also, when merging it into 4.2 and master branches, we needed to reword it entirely because the article is different in those branches and show the installation of the package (details: 7af99d4#diff-6e0f9a2335039341879e104549870a43).

Thanks for understanding!

@shreypuranik
Copy link
Contributor Author

@javiereguiluz Thanks! :)

xabbuh added a commit that referenced this pull request Mar 22, 2019
xabbuh added a commit that referenced this pull request Mar 22, 2019
* 3.4:
  [#11139] fix a minor typo
javiereguiluz added a commit to javiereguiluz/symfony-docs that referenced this pull request Mar 22, 2019
* 4.2:
  Add missing security guard sub-component
  Update unit_testing.rst
  [symfony#11139] fix a minor typo
  removed blank line and space
  add missing brackets
  use "behavior" instead of "behaviour"
  Remove unnecessary backslash escape
  Improve now what sentence
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