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

Issue 154 #246

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Issue 154 #246

wants to merge 6 commits into from

Conversation

Ansel-Hong
Copy link

Link to Github issue or other discussion

Replace this text - Provide links to any Github issues or other discussions that are related to this work.

What does thie PR do?

Replace this text - Explain the purpose/justification of this PR.

What changes were made?

Replace this text - State clearly the direct additions or modifications made in this pull request.

How to test / veryify this PR?

Replace this text - Document how to test this PR, and the details that help a reviewer verify the documentation. Include .yml configuration files and CSV data if possible. Unit or integration tests are also appreciated.

Interested Parties

Replace this text - name some folks who may be interested, or, if unsure, @mjordan


Checklist

  • Have you run pycodestyle --show-source --show-pep8 --ignore=E402,W504 --max-line-length=200 yourfile.py?
  • Have you included same configuration and/or CSV files useful for testing this PR?
  • Have you written unit or integration tests if applicable?
  • If the changes in this PR require an addiional Python library, have you included it in setup.py?
  • If the changes in this PR add a new configuration option, have you provided a default for when the option is not present in the .yml file?
  • Did the PR receive at least one approval from a committer, and all issues raised have been addressed?

@Ansel-Hong Ansel-Hong marked this pull request as draft March 17, 2021 03:41
@Ansel-Hong Ansel-Hong marked this pull request as ready for review March 24, 2021 05:57
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.

2 participants