-
Notifications
You must be signed in to change notification settings - Fork 556
Migrate Git commit cheat sheet #1513
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for moodledevdocs ready!Built without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR migrates the Git commit cheat sheet documentation from the old docs.moodle.org site to the dev docs, while also improving the documentation by removing redundant content and reorganizing sections.
Changes:
- Created new
/docs/guides/git/cheatsheet.mdfile with migrated commit cheat sheet content - Updated all internal references to point to the new location
- Moved "Introducing new strings" section to the Language strings section in Coding style documentation
- Added migration entry to
migratedPages.yml
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/guides/git/cheatsheet.md | New file containing the migrated Git commit cheat sheet with guidelines for commits, messages, authorship, AMOS scripts, and version changes |
| general/development/process/peer-review/index.md | Updated link reference from old docs.moodle.org URL to new internal path |
| general/development/policies/codingstyle/index.md | Updated link reference and added "Introducing new strings" subsection under Language strings |
| docs/guides/git/index.md | Updated "See also" link to use relative path to new cheatsheet file |
| data/migratedPages.yml | Added Commit_cheat_sheet entry with new filepath and slug |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
af7f9eb to
16dc580
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
* Remove the following:
- Testing instructions bit - It's already covered in
/general/development/process/testing
- Removing strings - Also already covered in the String deprecation
docs.
* Move the `Introducing new strings` section under the `Language`
section of the Coding style docs.
16dc580 to
1786239
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Migrate the Git commit cheat sheet documentation to the dev docs.
Also, some improvements:
/general/development/process/testingIntroducing new stringssection under theLanguagesection of the Coding style docs.