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

chore: remove trailing space in MD for stability banner #3188

Merged
merged 1 commit into from
Jul 3, 2019

Conversation

RomainMuller
Copy link
Contributor

The trailing space that was created when inserting an empty line in the
banner's message was very hsotile to manually editing the file with an
editor configured to prune trailing white space on save.

Added a trimRight() call to remove any trailing whitespace, so that
the banner never includes trailing whitespace, making the files nicer
to otherwise edit manually without risking to break pkglint compliance.


Please read the contribution guidelines and follow the pull-request checklist.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

The trailing space that was created when inserting an empty line in the
banner's message was very hsotile to manually editing the file with an
editor configured to prune trailing white space on save.

Added a `trimRight()` call to remove any trailing whitespace, so that
the banner never includes trailing whitespace, making the files nicer
to otherwise edit manually without risking to break pkglint compliance.
@RomainMuller RomainMuller requested a review from a team as a code owner July 3, 2019 12:50
@ghost ghost requested a review from shivlaks July 3, 2019 12:50
@eladb eladb merged commit 3a9fa64 into master Jul 3, 2019
@eladb eladb deleted the rmuller/no-trailing-white-space branch July 3, 2019 14:36
Kaixiang-AWS pushed a commit to Kaixiang-AWS/aws-cdk that referenced this pull request Jul 3, 2019
The trailing space that was created when inserting an empty line in the
banner's message was very hsotile to manually editing the file with an
editor configured to prune trailing white space on save.

Added a `trimRight()` call to remove any trailing whitespace, so that
the banner never includes trailing whitespace, making the files nicer
to otherwise edit manually without risking to break pkglint compliance.
@NGL321 NGL321 added the contribution/core This is a PR that came from AWS. label Sep 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contribution/core This is a PR that came from AWS.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants