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.

Sign up for GitHub

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

Jump to bottom

feat!: support unversioned data #27

Merged
merged 1 commit into from
Jan 31, 2024
Merged

feat!: support unversioned data #27

merged 1 commit into from
Jan 31, 2024

Conversation

jsstevenson
Copy link
Member

Useful for agct and cool-seq-tool

@jsstevenson jsstevenson added the priority:low Low priority label Jan 31, 2024
@korikuzma
Copy link
Member

@jsstevenson I think we should come up lab rules to determine who is assigned to a PR. I typically follow that it is the person who will ultimately be closing or merging the PR. I think instead I should be listed as a reviewer (I usually go here to see what PRs I need to review).

@jsstevenson
Copy link
Member Author

@korikuzma oh, that's my bad -- I think I accidentally hit the wrong dropdown (I think I do this a lot). agreed re: protocol

@jsstevenson jsstevenson merged commit d451254 into main Jan 31, 2024
10 checks passed
@jsstevenson jsstevenson deleted the issue-23 branch January 31, 2024 14:11
@jsstevenson jsstevenson mentioned this pull request Jan 31, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@korikuzma korikuzma korikuzma approved these changes

Assignees

@korikuzma korikuzma

Labels
priority:low Low priority
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@jsstevenson @korikuzma