Skip to content
This repository has been archived by the owner on Nov 3, 2023. It is now read-only.

Update conventions documentation for v4.0.0 #393

Merged
merged 5 commits into from Mar 7, 2020
Merged

Conversation

peterjc
Copy link
Contributor

@peterjc peterjc commented Aug 5, 2019

Addresses the problem I noted in a comment on issue #380

Thanks for submitting a PR!

Please make sure to check for the following items:

  • Add unit tests and integration tests where applicable.
    If you've added an error code or changed an error code behavior,
    you should probably add or change a test case file under tests/test_cases/ and add
    it to the list under tests/test_definitions.py.
    If you've added or changed a command line option,
    you should probably add or change a test in tests/test_integration.py.
  • Add a line to the release notes (docs/release_notes.rst) under "Current Development Version".
    Make sure to include the PR number after you open and get one.

Please don't get discouraged as it may take a while to get a review.

@peterjc
Copy link
Contributor Author

peterjc commented Aug 5, 2019

Added the command line API documentation update from #386 by @mrshannon

@samj1912
Copy link
Member

samj1912 commented Mar 2, 2020

Hey, sorry for the delay in reviews. If you are still interested, can you fix the merge conflicts and update the PR. Thanks!

@@ -66,6 +66,7 @@ Bug Fixes

Bug Fixes

* Update convention support documentation (#386, #393)
Copy link
Member

Choose a reason for hiding this comment

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

Looks like you updated the old release notes. Can you add it to the current development version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done. In hindsight I am surprised that wasn't part of the merge conflict.

@samj1912 samj1912 merged commit 5e96487 into PyCQA:master Mar 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants