moodle / devdocs

Source of the Moodle Developer Resources and Documentation
https://moodledev.io/
Other
36 stars 276 forks source link

Rename as many as possible "master" occurrences to "main" #997

Closed stronk7 closed 3 months ago

stronk7 commented 3 months ago

That includes:

We have kept apart a few corresponding to:

Also, a few things found, worth being considered:

  1. Suggestion: Maybe we could add some new test in order to detect any URL using the old "master" branch and failing on that, possible candidates are URLs like:
  2. While working on this I saw that there were a few links to the wiki docs of pages that already have been migrated. I've fixed them as part of the commit, but there should be some process detecting them at regular intervals at very least. The changes I applied were:
  3. Related with the previous point, are the internal links like the ones above checked? I ask because I did a typo in one of the 2 conversions above and the pre-commit hook did not detect it... maybe only GH workflows do that?
  4. I just saw the the jsdocs api docs are still using "master" instead of "main", surely we should make the switch there.

And that's all. I've performed one more search and all the remaining cases of "master" are justified, as explained above.

Again, please, don't forget about general/development/tools/mdk.md I've left it unmodified because it was not clear to me which "master" occurrences have to / can be replaced and which ones cannot. Friendly pinging to @junpataleta about that.

Ciao :-)

netlify[bot] commented 3 months ago

Deploy Preview for moodledevdocs ready!

Name Link
Latest commit 1e00687b9be4ef4af3e382ac3c39239156b3131b
Latest deploy log https://app.netlify.com/sites/moodledevdocs/deploys/663253ecf47fe00008919406
Deploy Preview https://deploy-preview-997--moodledevdocs.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

github-actions[bot] commented 3 months ago

⚡️ Lighthouse report for the deploy preview of this PR

URL Performance Accessibility Best Practices SEO PWA Report
/ 🟠 71 🟢 95 🟢 100 🟢 90 🟢 100 Report
/docs/4.4/apis/commonfiles 🟠 77 🟢 93 🟢 100 🟢 100 🟢 100 Report
/general/development/gettingstarted 🟠 66 🟢 95 🟢 100 🟢 90 🟢 100 Report
/general/releases 🟠 80 🟢 95 🟢 100 🟢 100 🟢 100 Report