learnxinyminutes-docs icon indicating copy to clipboard operation
learnxinyminutes-docs copied to clipboard

[MANY/en] Remove Trailing Whitespace

Open Kyle-Mendes opened this issue 5 years ago • 6 comments

  • [x] I solemnly swear that this is all original content of which I am the original author
  • [x] Pull request title is prepended with [language/lang-code]
  • [x] Pull request touches only one file (or a set of logically related files with similar changes made)
  • [x] Content changes are aimed at intermediate to experienced programmers (this is a poor format for explaining fundamental programming concepts)
  • [x] If you've changed any part of the YAML Frontmatter, make sure it is formatted according to CONTRIBUTING.md
    • [x] Yes, I have double-checked quotes and field names!

Kyle-Mendes avatar Oct 27 '19 17:10 Kyle-Mendes

@adambard I'm a little conflicted about merging this PR with a single PR changing 67 files. Any thoughts?

divayprakash avatar Jan 24 '20 14:01 divayprakash

@Kyle-Mendes Sorry for the delay in the review, I'm looking into this now.

divayprakash avatar Jan 24 '20 14:01 divayprakash

Oh hey sorry! Didn't see this notification because I was on vacation.

I'd be happy to split it up however you wanted. I just didn't want to spam the PR tab with a bunch of PRs all doing the same thing (just trimming trailing white space)

Just let me know and I can fix those conflicts.

Kyle-Mendes avatar Feb 13 '20 15:02 Kyle-Mendes

@Kyle-Mendes No issues! I'm blocking a merge on this for the time being, we're currently trying to push #3856 through and your PR affects the python docs. I assure you we will get back to this as soon as possible.

I'd be happy to split it up however you wanted. I just didn't want to spam the PR tab with a bunch of PRs all doing the same thing (just trimming trailing white space)

Thanks for this, I will keep in touch with you and let you know how to proceed.

divayprakash avatar Feb 13 '20 16:02 divayprakash

@Kyle-Mendes You can go ahead and fix the conflicts now and I'll merge this in. I imagine the python docs would need the most looking into.

divayprakash avatar Feb 14 '20 17:02 divayprakash

Hello, @Kyle-Mendes. Are you still interested in working on this? If you can fix the conflicts, I'll be happy to merge. If you don't have the time or are not interested anymore, this is fine. Just let me know so that I can close the PR.

mribeirodantas avatar Jul 08 '22 14:07 mribeirodantas

This has diverged too far, sorry.

vendethiel avatar Dec 14 '23 14:12 vendethiel