Thanks to visit codestin.com
Credit goes to github.com

Skip to content

MAINT: Revert #13876, "MAINT,BUG,DOC: Fix errors in _add_newdocs" #13943

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

Merged
merged 1 commit into from
Jul 9, 2019

Conversation

eric-wieser
Copy link
Member

@eric-wieser eric-wieser commented Jul 9, 2019

Reverts #13876.

Doing this so that I can reapply it without a squash commit, which will result in the github page at b592e8f showing master as one of the branches this fix is present in.

This only works if done as two separate PRs.

@charris
Copy link
Member

charris commented Jul 9, 2019

ISTR having that problem with a squash commit :)

@charris
Copy link
Member

charris commented Jul 9, 2019

Although I'm not convinced it is worth the hassle just to clean that up.

@charris charris changed the title Revert "MAINT,BUG,DOC: Fix errors in _add_newdocs" MAINT: Revert #13876, "MAINT,BUG,DOC: Fix errors in _add_newdocs" Jul 9, 2019
@eric-wieser
Copy link
Member Author

eric-wieser commented Jul 9, 2019

Hassle is minimal if I merge my own no-op PRs ;)

@mattip mattip merged commit 3085cb7 into master Jul 9, 2019
@mattip
Copy link
Member

mattip commented Jul 9, 2019

@eric-wieser sorry for the mess

@eric-wieser
Copy link
Member Author

@mattip: No worries, it's easy to fix

@eric-wieser
Copy link
Member Author

@charris: good guess, this was not worth it, and actually made things worse...

Before:

image

After:

image

Going to contact github and ask if this is intended behavior

@seberg seberg deleted the revert-13876-forbid-_add_newdocs-python branch August 14, 2019 19:12
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