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

Point release for peppy #301

Merged
merged 58 commits into from
May 2, 2019
Merged

Point release for peppy #301

merged 58 commits into from
May 2, 2019

Conversation

nsheff
Copy link
Contributor

@nsheff nsheff commented May 2, 2019

Due to the recent inadvertent caravel release we need to release peppy to support it.

Actually, probably good timing anyway.

vreuter and others added 2 commits May 2, 2019 09:59
@nsheff nsheff requested a review from vreuter May 2, 2019 17:39
@stolarczyk stolarczyk mentioned this pull request May 2, 2019
@vreuter
Copy link
Member

vreuter commented May 2, 2019

OK let me do ubiquerg 0.0.3

@nsheff
Copy link
Contributor Author

nsheff commented May 2, 2019

caravel -> looper -> peppy -> ubiquerg.

funny

@vreuter
Copy link
Member

vreuter commented May 2, 2019

OK ubiquerg has been released


### Added
- Support for Snakemake projects

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is there anything else we need to add to changelog @vreuter for this release?

Copy link
Member

Choose a reason for hiding this comment

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

Hmm let me double-check

grab_project_data, parse_ftype, sample_folder
from ngstk import peek_read_lengths_and_paired_counts_from_bam
Copy link
Contributor Author

Choose a reason for hiding this comment

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

the new ngstk is showing up here already!

that is quite a function name!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

wow you've already got it on pypi. cool.

Copy link
Member

Choose a reason for hiding this comment

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

I couldn't come up with anything both meaningful and short...check_bam made a lot of sense when used more locally, but I wanted a more descriptive name for export from the seq. utils package

Copy link
Member

Choose a reason for hiding this comment

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

OK 1ee0025 notes some improvements and changes

@nsheff nsheff requested a review from stolarczyk May 2, 2019 20:50
@nsheff
Copy link
Contributor Author

nsheff commented May 2, 2019

@MichalStolarczyk ready for this?

Copy link
Member

@stolarczyk stolarczyk left a comment

Choose a reason for hiding this comment

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

ready

@nsheff nsheff merged commit d2c2349 into master May 2, 2019
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