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

test/sharness: improve t0080 #716

Merged
merged 1 commit into from Feb 2, 2015
Merged

test/sharness: improve t0080 #716

merged 1 commit into from Feb 2, 2015

Conversation

chriscool
Copy link
Contributor

The following changes are made to make t0080
cleaner, safer and more canonical:

  • remove useless stuff,
  • don't use a pipe after ipfs commands,
  • use test_must_fail before ipfs commands instead of !

License: MIT
Signed-off-by: Christian Couder chriscool@tuxfamily.org

The following changes are made to make t0080
cleaner, safer and more canonical:

- remove useless stuff,
- don't use a pipe after ipfs commands,
- use test_must_fail before ipfs commands instead of !

License: MIT
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
@btc btc added the status/in-progress In progress label Feb 2, 2015
@whyrusleeping
Copy link
Member

@chriscool you are a wizard

@chriscool
Copy link
Contributor Author

@whyrusleeping thanks!

@jbenet
Copy link
Member

jbenet commented Feb 2, 2015

LGTM!

jbenet added a commit that referenced this pull request Feb 2, 2015
test/sharness: improve t0080
@jbenet jbenet merged commit 4539b97 into master Feb 2, 2015
@jbenet jbenet deleted the improve_t0080 branch February 2, 2015 02:33
@jbenet jbenet removed the status/in-progress In progress label Feb 2, 2015
@aschmahmann aschmahmann mentioned this pull request Aug 23, 2021
62 tasks
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.

None yet

4 participants