-
Notifications
You must be signed in to change notification settings - Fork 14
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
README: document usage #102
Conversation
@hacdias you've been working on the test suite as well, you might have feedback about this :) |
Makefile
Outdated
./gateway-conformance test --verbose --json reports/output.json --gateway-url http://127.0.0.1:8080 --subdomain-url http://example.com:8080 | ||
|
||
test-kubo: provision-kubo gateway-conformance | ||
./gateway-conformance test --json reports/output.json --gateway-url http://127.0.0.1:8080 --specs -subdomain-gateway | ||
./gateway-conformance test --verbose --json reports/output.json --gateway-url http://127.0.0.1:8080 --specs -subdomain-gateway |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we want to make verbose output the default? By this point it's a loooooot.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No strong opinion here, reverted!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is a good starting point and addition to the README. In my personal case, I usually already have everything configured - as I usually also create the fixtures -, so my process is often different 😅
Contributes to #8