Skip to content

Commit

Permalink
[docs] Fix all uses of apostrophe vs. single-quote
Browse files Browse the repository at this point in the history
  • Loading branch information
andrewnicols committed May 4, 2022
1 parent a390a29 commit 5fcdec0
Show file tree
Hide file tree
Showing 4 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion general/community/roadmap.md
Expand Up @@ -36,7 +36,7 @@ Working on the Moodle learning platform involves millions of moving parts, and e

The entire user experience from onboarding, into daily teaching/learning and expert customisation of Moodle is the core value of what makes Moodle useful or not in the real world, in fully online and blended modes.

While were working on hundreds of smaller, annoying issues, we are also doing some major re-thinking around what an LMS should be in the next decade and beyond as a tool to empower educators and learners.
While we're working on hundreds of smaller, annoying issues, we are also doing some major re-thinking around what an LMS should be in the next decade and beyond as a tool to empower educators and learners.

#### Enabling all our developers

Expand Down
4 changes: 2 additions & 2 deletions general/development/policies/accessibility.md
Expand Up @@ -198,11 +198,11 @@ Moodle aims to comply with the following international legislation where possibl

<!-- cspell:ignore Référentiel -->
<!-- cspell:ignore Général -->
<!-- cspell:ignore dAccessibilité -->
<!-- cspell:ignore d'Accessibilité -->
<!-- cspell:ignore des -->
<!-- cspell:ignore RGAA -->

- [Référentiel Général dAccessibilité des Administrations - RGAA](https://references.modernisation.gouv.fr/rgaa-accessibilite/)
- [Référentiel Général d'Accessibilité des Administrations - RGAA](https://references.modernisation.gouv.fr/rgaa-accessibilite/)

### European Union

Expand Down
2 changes: 1 addition & 1 deletion general/development/process/peer-review.md
Expand Up @@ -311,7 +311,7 @@ If some additional work is needed or the author specifically asked not to submit

When the code has come from an external developer, the peer reviewer will also help the developer to lead the issue to integration. In this case the peer reviewer should not use “Finish peer review” button.

If the issue needs additional work, the peer reviewer comments about the suggested changes but does not change the status of the issue and it remains as “Peer review in progress”. If the author of the patch does not reply in 4 days, the peer reviewer may select either to complete the patch themselves or reopen the issue. The decision should be based on the amount of work required to complete the solution, for example, changing coding style or commit message, rebasing, backporting, adding testing instructions, etc. should be performed by the peer reviewer. This may require picking the commits into reviewers git branch.
If the issue needs additional work, the peer reviewer comments about the suggested changes but does not change the status of the issue and it remains as “Peer review in progress”. If the author of the patch does not reply in 4 days, the peer reviewer may select either to complete the patch themselves or reopen the issue. The decision should be based on the amount of work required to complete the solution, for example, changing coding style or commit message, rebasing, backporting, adding testing instructions, etc. should be performed by the peer reviewer. This may require picking the commits into reviewer's git branch.

It is very important to give the credit to the author of the code by either keeping their authorship on the commit or adding “Thanks to XXXX for providing the patch” to the commit message. If the author of the patch is not in jira-developers group the special user 'moodle.com' should be entered in Assignee field.

Expand Down
4 changes: 2 additions & 2 deletions general/development/process/testing/qa.md
Expand Up @@ -190,7 +190,7 @@ To create a new QA test:
5. In the description field add the test steps (usually between 3 and 10), similar to the issue's [testing instructions](./guide), starting with 'Log in as a teacher...' or similar. It's a good idea to try doing the steps yourself as you write the test.
6. Start some steps with 'Verify that ...' or similar.
7. Click the Create button.
8. Go to the MDL issue and create a ‘has a QA test link to the new QA test, adding a comment “This feature is now covered by the QA test MDLQA....”.
8. Go to the MDL issue and create a ‘has a QA test' link to the new QA test, adding a comment “This feature is now covered by the QA test MDLQA....”.

:::note

Expand All @@ -199,7 +199,7 @@ To create a new QA test:
- For OAuth 2 tests and any other tests which require a client ID or secret to be entered, label `credentials_required`.
- For issues which specifically mention in the testing instructions to test in different browsers, use the phrase:

> Test in as many browsers as possible and mention in a comment which ones youve used.
> Test in as many browsers as possible and mention in a comment which ones you've used.
- For an exploratory test, begin the test description with:

Expand Down

0 comments on commit 5fcdec0

Please sign in to comment.