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

style: Remove trailing space characters #651

Merged
merged 1 commit into from
Apr 26, 2022

Conversation

striezel
Copy link
Contributor

Description

Removes unnecessary trailing space characters in .cpp/.hpp files.

No functional changes, this is just a "nice to have" cleanup thing.

References

None.

Tasklist

  • Ensure all CI builds pass
  • Review and approve

@striezel
Copy link
Contributor Author

Azure Pipelines with Windows 2016 failed. To be more precise, it was cancelled on start. It shows the following error:

This is a scheduled windows-2016 brownout. The windows-2016 environment is deprecated and will be removed on April 1st, 2022. For more details, see actions/runner-images#5238

Looks like Microsoft is finally removing Windows 2016 / Visual Studio 2019 there, too, just like they did on the GitHub Actions environments a while ago. (Also see https://devblogs.microsoft.com/devops/hosted-pipelines-image-deprecation/.) So this means I will probably send another PR for the CI soon, this time updating or removing Windows 2016 stuff of Azure.

@mloskot
Copy link
Member

mloskot commented Apr 26, 2022

I will probably send another PR for the CI soon, this time updating or removing Windows 2016 stuff of Azure.

👍 Thanks in advance!

@mloskot mloskot changed the title remove trailing space characters style: Remove trailing space characters Apr 26, 2022
@mloskot mloskot merged commit 4f83beb into boostorg:develop Apr 26, 2022
@striezel striezel deleted the remove-trailing-spaces branch April 26, 2022 19:19
@mloskot mloskot mentioned this pull request May 12, 2022
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants