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

Sam parser #5

Merged
merged 24 commits into from
Jan 1, 2024
Merged

Sam parser #5

merged 24 commits into from
Jan 1, 2024

Conversation

Koeng101
Copy link
Owner

@Koeng101 Koeng101 commented Dec 7, 2023

forked from bebop/poly#333

Koeng101 and others added 15 commits August 16, 2023 18:43
@Koeng101
minimally functioning sam parser
cddd21c
@Koeng101 @carreter
Update io/sam/sam.go
2d9aee3 
Co-authored-by: Willow Carretero Chavez <[email protected]>
@Koeng101
few udpates from comments
eeb47ff
@Koeng101
Merge branch 'samParser' of github.com:TimothyStiles/poly into samParser
1a3dc1d
@Koeng101 @carreter
Update io/sam/sam.go
e46f5e1 
Co-authored-by: Willow Carretero Chavez <[email protected]>
@Koeng101
Merge branch 'samParser' of github.com:TimothyStiles/poly into samParser
27015f7
@Koeng101 @carreter
Update io/sam/sam.go
227e59e 
Co-authored-by: Willow Carretero Chavez <[email protected]>
@Koeng101
add validate
ab68b0d
@Koeng101
Merge branch 'samParser' of github.com:TimothyStiles/poly into samParser
0127dd0
@Koeng101 @carreter
Update io/sam/sam.go
2b5ac83 
Co-authored-by: Willow Carretero Chavez <[email protected]>
@Koeng101
Merge branch 'samParser' of github.com:TimothyStiles/poly into samParser
f8161f0
@Koeng101
fixed sam_test
efe6402
@Koeng101
Merge branch 'main' into samParser
7f75ef1
@Koeng101
Add in bio parser
7be4933
@Koeng101
Updated sam to be in bio
cdf61aa
@Koeng101
Copy link
Owner Author

Need to write the WriteTo functions, get much better test coverage, and then document the hell out of the package, including a large context package doc.

@Koeng101
Copy link
Owner Author

The only thing left to do is get full test coverage.

@Koeng101 Koeng101 merged commit 2203bad into main Jan 1, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Koeng101