Skip to content
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

Modified the PR template to include README.md update reqirement if ne… #3303

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

bgorski
Copy link

@bgorski bgorski commented Apr 2, 2021

…cessary

Description (*)

With the introduction of the README.md template, there is a requirement of describing module installation, structure and extension points in the readme.
We need to pay special attention to README.md always being up to date, because an outdated readme can do more harm than good. This is why all code contributors need to be aware that they are supposed to modify README.md as well when they create modifications that affect any of readme predefined sections.
Also, because this repository didn't have the contribution checklist section at all, I added it.

Fixed Issues (if relevant)

  1. Fixes magento/inventory#<issue_number>: Issue title

Manual testing scenarios (*)

  1. Attempt to create a Pull Request to this or any other github repository
  2. When on the PR edit page, copy-paste the modified template content into your PR description area
  3. Click Preview
  4. Along the Contribution checklist checkboxes you should see a new checkbox with a description saying that README.md needs to be modified if necessary
  5. Check if it looks good
  6. Don't create that PR as you don't need to, unless you want to check that the markdown looks the same on an actual PR as in the Preview window

Questions or comments

Contribution checklist (*)

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/integration tests (if applicable)
  • All automated tests passed successfully (all builds are green)

@bgorski
Copy link
Author

bgorski commented Apr 2, 2021

@magento run all tests

Stepa4man
Stepa4man previously approved these changes Apr 20, 2021
@m2-community-project m2-community-project bot moved this from Ready for Review to Reviewer Approved in Pull Request Progress Apr 20, 2021
@sidolov sidolov changed the base branch from 1.2-develop to develop November 11, 2021 17:47
@sidolov sidolov dismissed Stepa4man’s stale review November 11, 2021 17:47

The base branch was changed.

@m2-community-project m2-community-project bot moved this from Reviewer Approved to Review in Progress in Pull Request Progress Nov 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Pull Request Progress
  
Review in Progress
Development

Successfully merging this pull request may close these issues.

None yet

3 participants