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

add nucl mappings, slight reorg #80

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

Conversation

bluegenes
Copy link
Collaborator

Many thanks to contributing to czbiohub/sencha!

Please fill in the appropriate checklist below (delete whatever is not relevant). These are the most common things requested on pull requests (PRs).

PR checklist

  • This comment contains a description of changes (with reason)
  • If you've fixed a bug or added code that should be tested, add tests!
  • Ensure the test suite passes with pytest . (command to run: pytest or make coverage if you want to see which lines don't have tests yet)
  • Make sure your code is linted and autoformatted using black (black . --check).
  • Documentation in usage.md is updated
  • README.md is updated

@olgabot
Copy link
Contributor

olgabot commented Jul 7, 2020

Thank you! Sorry to be nitpicky, but could you add some tests for the functions that are in this PR?

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.

2 participants