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

[DOC] missing commands in the online doc #461

Closed
Remi-Gau opened this issue Aug 1, 2022 · 6 comments · Fixed by #463
Closed

[DOC] missing commands in the online doc #461

Remi-Gau opened this issue Aug 1, 2022 · 6 comments · Fixed by #463

Comments

@Remi-Gau
Copy link
Collaborator

Remi-Gau commented Aug 1, 2022

Not urgent but I think that some commands are not listed in the online doc.

For some reason I tend to go for the online doc rather that typing --help and I got confused by some things I knew should be there.

Comparing for example:

With:
docker run -it --rm repronim/neurodocker:0.7.0 generate docker --help

Missing things like

  • --entrypoint
  • --cmd
@kaczmarj
Copy link
Collaborator

kaczmarj commented Aug 1, 2022

thanks @Remi-Gau for pointing this out - the docs definitely need work. good to know that you are using the online doc rather than --help -- i'm sure many others do the same. given that, docs need to become a priority.

thanks for the missing items. to anyone reading this, pull requests are welcome of course. i'll try to address this issue in the coming days if no pull requests come in.

@Remi-Gau
Copy link
Collaborator Author

Remi-Gau commented Aug 1, 2022

I have not had a close look but I think that most of your doc must be auto generated through some sphinx / rst directives.

Not sure where the discrepancy is coming from. If that part of the doc is generated manually, then that is probably what needs to change. :-)

@kaczmarj
Copy link
Collaborator

kaczmarj commented Aug 1, 2022

yes that part of the doc is manually done. the API docs are auto-generated.

@satra
Copy link
Contributor

satra commented Aug 1, 2022

we may want to enable the click doc extension that we use in dandi cli: https://dandi.readthedocs.io/en/latest/

@kaczmarj
Copy link
Collaborator

kaczmarj commented Aug 1, 2022

@satra - thanks for the pointer. had not seen that before.

@github-actions
Copy link
Contributor

This issue is stale because it has been open for 30 days with no activity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants