Merged
Conversation
Corrected grammatical errors, style and typos. Please cross check before approving. Also i think in line 180 "The strategic value the project holds for the organization". Should be another point made in the points listed before the paragraph. Please revise correction if i am wrong. Signed-off-by: Harmony Elendu <164214092+harmonyelendu@users.noreply.github.com>
✅ Deploy Preview for ospobook ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ Deploy Preview for ospomindmap canceled.
|
Reverting as this changes the meaning and makes less sense. Signed-off-by: Alice Sowerby <acsowerby@hotmail.com> Signed-off-by: Alice Sowerby <acsowerby@hotmail.com>
Reverting as the style guide prefers contractions. Signed-off-by: Alice Sowerby <acsowerby@hotmail.com> Signed-off-by: Alice Sowerby <acsowerby@hotmail.com>
Signed-off-by: Alice Sowerby <acsowerby@hotmail.com> Signed-off-by: Alice Sowerby <acsowerby@hotmail.com>
Signed-off-by: Alice Sowerby <acsowerby@hotmail.com> Signed-off-by: Alice Sowerby <acsowerby@hotmail.com>
| * Support the process with training, tools, and regular team discussions. | ||
|
|
||
| By identifying problems early and responding quickly, organizations can reduce risks and make sure their open source software stays secure, stable, and sustainable. | ||
| By identifying problems early and responding quickly, organizations can reduce risks and make sure their open source software remains secure, stable, and sustainable. |
Contributor
There was a problem hiding this comment.
I don't know why this change is suggested. It makes little difference.
The link was wrong Signed-off-by: Alice Sowerby <acsowerby@hotmail.com> Signed-off-by: Alice Sowerby <acsowerby@hotmail.com>
Reverting as it is consistent with the title of the actual paper. Signed-off-by: Alice Sowerby <acsowerby@hotmail.com> Signed-off-by: Alice Sowerby <acsowerby@hotmail.com>
alice-sowerby
approved these changes
Apr 22, 2025
Contributor
alice-sowerby
left a comment
alice-sowerby
left a comment
There was a problem hiding this comment.
Thanks for the corrections, most are good and worth having.
Some feedback to make it easier on the reviewer of the PR:
Please do not submit changes that contradict the style guide. It's quite a pain to go through and change things back when we already took the trouble to change them just last week.
Also, changing the name of the paper without checking that it is correct is also extra work that could have been avoided.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Corrected grammatical errors, style and typos. Please cross check before approving.
Also i think in line 180
"The strategic value the project holds for the organization". Should be another point made in the points listed before the paragraph. Please revise correction if i am wrong.