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

Invocation #115

Closed
wants to merge 1 commit into from
Closed

Invocation #115

wants to merge 1 commit into from

Conversation

OvalAdmin
Copy link

This includes a terse indication of how to invoke the test suite and surreptitiously implies that one should set the --basetemp argument

Checklist

  • Pull request details were added to HISTORY.rst

This includes a terse indication of how to invoke the test suite and surreptitiously implies that one should set the `--basetemp` argument
@rhpvorderman
Copy link
Member

I do not think this is the correct place to make such an adjustment:

  1. Getting started is for explaining the basics. The default temp location is probably fine for most users.
  2. There is a usage section in the documentation that can be used to more elaborately explain options.

I agree that the --basetemp option is a bit hidden in a wall of text in the current documentation. Maybe giving it a separate header: "Changing the temporary directory" will make it more easy to find?

@OvalAdmin
Copy link
Author

You are right it was/is well marked in the documentation; I simply missed it the first time I read it.

I posted the PR as a suggestion more then anything else. It was only place I could see to propose an edit albeit a bit superfluous. I'll close this and you can disregard the submission.

Thanks for considering it though.

@OvalAdmin OvalAdmin closed this Jul 15, 2020
@OvalAdmin OvalAdmin deleted the patch-1 branch July 15, 2020 17:14
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

2 participants