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

GitHub Pull Request Template Update #927

Merged
merged 5 commits into from
May 2, 2020

Conversation

minchinweb
Copy link
Contributor

Further to #911, this aims to improve the Pull Request template, by including a reminder of the commands to run locally for test, lints, etc.

Checklist

  • The code change is tested and works locally.
  • [ n/a ] Tests pass. Your PR cannot be merged unless tests pass
  • There is no commented out code in this PR.
  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • [ n/a ] Have you written new tests for your core changes, as applicable?

jrnl-bot and others added 4 commits April 25, 2020 21:07
# Conflicts:
#	.github/PULL_REQUEST_TEMPLATE.md
pyflakes here is given only certain subdirectories because `.` will try and run across all packages installed in the virtual environment, if the virtual environment is within the project's root folder (which is my common practice).
wren
wren previously approved these changes May 2, 2020
Copy link
Member

@wren wren left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Thank you!

@wren wren merged commit 870c325 into jrnl-org:develop May 2, 2020
@wren
Copy link
Member

wren commented May 2, 2020

@minchinweb Thanks for this.

Out of curiosity, do you have make set up on your computer?

@minchinweb
Copy link
Contributor Author

No, I don't have make installed here (as, I think, is typical of Windows machines). (or bash for the matter...)

invoke might be a good pure-Python alternative that I've used in the past that might do what you want.

@minchinweb minchinweb deleted the github-pr-template branch May 3, 2020 04:14
@wren
Copy link
Member

wren commented May 4, 2020

That's great; I was actually hoping you didn't have it. Would you mind going through the contributing doc and getting set up? I'd love your notes on whether there's enough info there or not for the Windows setup.

@wren
Copy link
Member

wren commented May 7, 2020

Forgot to @minchinweb for the above comment.

wren pushed a commit that referenced this pull request Jul 25, 2020
* Improve GitHub PR Template
* suggest running commands with `poetry run ...`

pyflakes here is given only certain subdirectories because `.` will try and run across all packages installed in the virtual environment, if the virtual environment is within the project's root folder (which is my common practice).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants