Skip to content

DOC: Cleaner template for PRs - #17440

Merged
charris merged 3 commits into
numpy:masterfrom
bjnath:bolder-PR-template
Oct 4, 2020
Merged

charris merged 3 commits into
numpy:masterfrom
bjnath:bolder-PR-template

Conversation

@bjnath

@bjnath bjnath commented Oct 3, 2020

Copy link
Copy Markdown
Contributor

Make the instructions easy to see and hard to delete unread.

Moves some text to new sections in dev/development_workflow.rst.

The dashes above and below the heading, which in monospace seem too long, fit it properly in Chromium browsers and reasonably in Firefox.

Make the instructions easy to see and hard to delete unread.

Moves some text to new sections in dev/development_workflow.rst.

The dashes above and below the heading, which
in monospace seem too long, fit it
well in Chromium browsers and reasonably in Firefox.
bjnath added 2 commits October 3, 2020 17:11
Colon missing in heading.

Added sphinx anchor for PR-timeline (html anchor was OK).
@charris
charris merged commit 8182943 into numpy:master Oct 4, 2020
@charris

charris commented Oct 4, 2020

Copy link
Copy Markdown
Member

Let's give this a shot. Thanks @bjnath .

@mattip

mattip commented Oct 8, 2020

Copy link
Copy Markdown
Member

We may want to reduce the amount of whitespace by deleting blank lines. It probably won't help, but maybe some contributors will read the text if they don't have to scoll too far.

@bjnath

bjnath commented Oct 8, 2020

Copy link
Copy Markdown
Contributor Author

@mattip I was thinking the exact same thing.

bjnath added a commit to bjnath/numpy-1 that referenced this pull request Oct 8, 2020
Reduce the need to scroll, per @mattip suggestion in numpy#17440
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants