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

Set up CI and add tests #1

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
1 participant
@luizirber
Copy link

commented Jun 8, 2019

  • Remove extern crate (not needed in the 2018 edition)
  • Rustfmt and clippy fixes
  • Add basic test and demo for data

The test doesn't do anything because I don't know exactly what to check yet, but at least they are testing if the arguments are being passed correctly and the commands run.

@luizirber luizirber force-pushed the luizirber:ci_and_tests branch from 59e854c to e12201e Jun 8, 2019

Set up CI and add tests
- Remove `extern crate` (not needed in the 2018 edition)
- Rustfmt and clippy fixes
- Add basic test and demo for data

@luizirber luizirber force-pushed the luizirber:ci_and_tests branch from e12201e to 335756c Jun 11, 2019

@@ -366,20 +376,22 @@ fn main() -> io::Result<()> {
SubCommand::with_name("compare")
.arg(
Arg::with_name("gfa1")
.short("g1")
.short("1")

This comment has been minimized.

Copy link
@luizirber

luizirber Jun 11, 2019

Author

I changed the name of the arguments because short args can't have more than one letter

@luizirber

This comment has been minimized.

Copy link
Author

commented Jun 11, 2019

Also: can you publish your rust-debruijn changes in a fork and branch in your account? This way cargo can download from the branch and avoid build errors like this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.