Skip to content

Conversation

@Preocts
Copy link
Owner

@Preocts Preocts commented Jun 22, 2025

The write process was creating an extra newline at the end of the file. Now it won't do that.

If the input file does not end with a newline, the written result will. As all files should.

The write process was creating an extra newline at the end of the file.
Now it won't do that.

If the input file does not end with a newline, the written result will.
As all files should.
@Preocts Preocts self-assigned this Jun 22, 2025
@Preocts Preocts marked this pull request as ready for review June 22, 2025 18:47
@Preocts Preocts merged commit eac76bd into main Jun 22, 2025
16 checks passed
@Preocts Preocts deleted the correct-end-of-file branch June 22, 2025 18:47
Preocts added a commit that referenced this pull request Jun 22, 2025
resolves #68

This change treats emtpy lines as comments allowing the preservation of
them from read to write.

The changes in #73 are removed here. #73 specifically corrected the
ending of the file. Instead, this change takes full control of empty
lines. This corrects the issue #73 solved as well as correcting extra
empty lines that were appearing between sections during development.
Preocts added a commit that referenced this pull request Jun 22, 2025
* Preserve blank line formatting

resolves #68

This change treats emtpy lines as comments allowing the preservation of
them from read to write.

The changes in #73 are removed here. #73 specifically corrected the
ending of the file. Instead, this change takes full control of empty
lines. This corrects the issue #73 solved as well as correcting extra
empty lines that were appearing between sections during development.

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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