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

Error docs related to parallelization should have link to parallelization docs #974

Closed
Bkucera opened this Issue Sep 19, 2018 · 2 comments

Comments

@Bkucera
Member

Bkucera commented Sep 19, 2018

Every error related to parallelization should have a reference to parallelization docs

I don't think the --parallel should be linked, because that simply leads users to the command line reference, which gives very little info

image

@steakfrye

This comment has been minimized.

Show comment
Hide comment
@steakfrye

steakfrye Sep 20, 2018

Contributor

I'm interested in implementing this, but I want to make sure I understand the issue. Let me know if this is correct and I'll start working on it.

The issue is on the errors page. Instead of linking here where the flag is not explained, it should point users to the guide to actually understand parallelization.
Is that right?

Contributor

steakfrye commented Sep 20, 2018

I'm interested in implementing this, but I want to make sure I understand the issue. Let me know if this is correct and I'll start working on it.

The issue is on the errors page. Instead of linking here where the flag is not explained, it should point users to the guide to actually understand parallelization.
Is that right?

@Bkucera

This comment has been minimized.

Show comment
Hide comment
@Bkucera

Bkucera Sep 21, 2018

Member

@steakfrye Thanks! I think we can take off the links from the --parallel and include blurbs on each error to visit parallelization guide

Member

Bkucera commented Sep 21, 2018

@steakfrye Thanks! I think we can take off the links from the --parallel and include blurbs on each error to visit parallelization guide

@steakfrye steakfrye referenced this issue Sep 23, 2018

Closed

Master #977

@amirrustam amirrustam modified the milestones: Sprint 3, Sprint 4 Sep 24, 2018

@steakfrye steakfrye referenced this issue Sep 25, 2018

Merged

Fixes #984

@jennifer-shehane jennifer-shehane self-assigned this Sep 27, 2018

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