Skip to content

Conversation

@apt1002
Copy link
Collaborator

@apt1002 apt1002 commented May 3, 2025

Update and fix Cookbook.md.

@rrthomas
Copy link
Owner

rrthomas commented May 3, 2025

This looks good. Please could you explain in the commit message why a bunch of test results changed, and also separate out the formatting fix to README.nancy.md?

@apt1002
Copy link
Collaborator Author

apt1002 commented May 4, 2025

I think you must be referring to the commit remove trailing newlines from index.html` files.

One of those files is included into Cookbook.md where the trailing newline is rendered as an unwanted space. This seemed like an easy way to fix the problem. Perhaps there's a batter way?

@apt1002
Copy link
Collaborator Author

apt1002 commented May 4, 2025

I restored the trailing newlines on index.html files, and instead put a head -c-1 command into Cookbook.nancy.md to remove the unwanted newline character. This avoids any changes to the test files.

@rrthomas rrthomas merged commit a15d1f3 into main May 4, 2025
4 checks passed
@apt1002 apt1002 deleted the cookbook branch May 4, 2025 12:05
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.

3 participants