Skip to content

Comments

Change Readme to Markdown #446#447

Merged
TobiasNx merged 3 commits intomasterfrom
446-readmeMarkdown
Feb 9, 2026
Merged

Change Readme to Markdown #446#447
TobiasNx merged 3 commits intomasterfrom
446-readmeMarkdown

Conversation

@TobiasNx
Copy link
Contributor

@TobiasNx TobiasNx commented Nov 26, 2025

Replace textile with markdown and add new headline and adjust other heading order.

Copy link
Contributor

@acka47 acka47 left a comment

Choose a reason for hiding this comment

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

+1

README.md Outdated
@@ -1,76 +1,78 @@
h1. About
#README
Copy link
Member

Choose a reason for hiding this comment

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

This needs a blank to be a header, but I don't think it makes much sense. I think you meant to add that as a new main header and moved all other header down one level. I think the levels and headers should remain as before.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I reword the commit and adjusted the heading order in a new separate commit.

@fsteeg fsteeg assigned TobiasNx and unassigned fsteeg and acka47 Nov 26, 2025
@acka47 acka47 linked an issue Dec 1, 2025 that may be closed by this pull request
@TobiasNx TobiasNx assigned fsteeg and unassigned TobiasNx Feb 9, 2026
@TobiasNx TobiasNx requested a review from fsteeg February 9, 2026 11:01
Copy link
Member

@fsteeg fsteeg left a comment

Choose a reason for hiding this comment

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

I still don't see how moving the headers one level down improves anything, but won't die on that hill.

Branches that are under review should not be force-pushed.

Updated two 404 links in 8fe1e31.

Switching to Markdown is great, so thanks for that.

@TobiasNx
Copy link
Contributor Author

TobiasNx commented Feb 9, 2026

I am refering th the google markdown style guide with regard to the headings in the nwbib repo: hbz/nwbib#672 (comment)

@TobiasNx TobiasNx merged commit 7b2c4ff into master Feb 9, 2026
1 check passed
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.

Change Readme to Markdown

3 participants