Skip to content

Remove trailing whitespace#231

Merged
ChrisThrasher merged 1 commit intomasterfrom
trailing_whitespace
Feb 19, 2024
Merged

Remove trailing whitespace#231
ChrisThrasher merged 1 commit intomasterfrom
trailing_whitespace

Conversation

@ChrisThrasher
Copy link
Copy Markdown
Member

@ChrisThrasher ChrisThrasher commented Feb 19, 2024

Easy first step towards fixing the inconsistent formatting before we eventually (hopefully) get clang-format running in CI.

@ChrisThrasher ChrisThrasher merged commit e67340f into master Feb 19, 2024
@ChrisThrasher ChrisThrasher deleted the trailing_whitespace branch February 19, 2024 22:55
@ChrisThrasher ChrisThrasher removed the request for review from eXpl0it3r February 19, 2024 22:55


* `build.win.ps1`
* A [PowerShell](https://github.com/PowerShell/PowerShell) script to build Windows native libraries. By default, both `win-x86` and `win-x64` are built. To build only one specific Runtime Identifier, pass it to the script as a parameter (`./build.win.ps1 win-x64`)
Copy link
Copy Markdown
Contributor

@kimci86 kimci86 Feb 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Two trailing whitespaces mean a linebreak in markdown.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Related discussion: #218 (comment)

Copy link
Copy Markdown
Member

@eXpl0it3r eXpl0it3r Feb 19, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep

Before:

image

After:

image

It's not really bad here, but I think kimci86 mentioned that on another PR once before. Something to keep in mind when "formatting" markdown files.

Edit: Got 🥷-ed with that comment link 😄

@eXpl0it3r eXpl0it3r added this to the 3.0 milestone Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants