Skip to content

DOCS: Add final newline to markdown files#19645

Merged
MrJul merged 1 commit into
AvaloniaUI:masterfrom
VbhvGupta:master
Sep 15, 2025
Merged

DOCS: Add final newline to markdown files#19645
MrJul merged 1 commit into
AvaloniaUI:masterfrom
VbhvGupta:master

Conversation

@VbhvGupta

Copy link
Copy Markdown
Contributor

This PR adds a missing final newline character to several documentation files, including api-compat.md and build.md.

Adding a final newline is a standard POSIX convention for text files. This change improves compatibility with various command-line tools and helps keep Git diffs clean and easy to read.

The POSIX standard defines a Line as:

This means that for a sequence of characters to be considered a complete "line" according to the standard, it must end with a newline. A file that lacks a final newline has an incomplete last line and is therefore not a strictly compliant POSIX text file.

Several documentation files were missing a final newline character.
This change adds them to adhere to POSIX conventions and improve
compatibility with various text-processing tools. It also helps
to create cleaner diffs in the future.
@avaloniaui-bot

Copy link
Copy Markdown

You can test this PR using the following package version. 12.0.999-cibuild0058870-alpha. (feed url: https://nuget-feed-all.avaloniaui.net/v3/index.json) [PRBUILDID]

@cla-avalonia

cla-avalonia commented Sep 14, 2025

Copy link
Copy Markdown
Collaborator
  • All contributors have signed the CLA.

@MrJul

MrJul commented Sep 14, 2025

Copy link
Copy Markdown
Member

Note: we do have insert_final_newline in .editorconfig for all files, but these markdown files were probably edited with an editor that doesn't enforce that.

@MrJul MrJul left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, thank you!

@MrJul

MrJul commented Sep 14, 2025

Copy link
Copy Markdown
Member

(Waiting for the CLA to be signed before merging)

@VbhvGupta

VbhvGupta commented Sep 15, 2025

Copy link
Copy Markdown
Contributor Author

@cla-avalonia agree

@MrJul MrJul added this pull request to the merge queue Sep 15, 2025
Merged via the queue into AvaloniaUI:master with commit f4e653a Sep 15, 2025
12 checks passed
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.

4 participants