Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use github compatible markdown for README #163

Open
bevsxyz opened this issue May 13, 2022 · 8 comments
Open

Use github compatible markdown for README #163

bevsxyz opened this issue May 13, 2022 · 8 comments

Comments

@bevsxyz
Copy link

bevsxyz commented May 13, 2022

The current README file is not formatted for GitHub. Change the formatting to markdown with the necessary changes to formatting.

@MichaIng
Copy link
Contributor

Might be fine, but just to widen the view:

Remember GitHub is not the only/main target for this README, but it is included in Dropbear sources downloads from the official website/server: https://matt.ucc.asn.au/dropbear/releases/

When downloading such an archive, Markdown is probably not the format wanted for an universally readable README, especially not a possibly non-standard GitHub-flavoured Markdown format with external resources embedded.

@bkil
Copy link

bkil commented Dec 5, 2022

There is no such thing as "standard" markdown - there exist multiple dialects. Certain subsets of features of them do intersect, however. And if you stick to adding extra whitespace and stay away from advanced features, it should look the same on all output.

I can read downloaded markdown files offline just fine, but if this bothers you, the readme could be separate to a shiny part that appears on the GitHub main page and a more dull (but still markdown) one that contains instructions for installation, usage and whatnot.

@MichaIng
Copy link
Contributor

MichaIng commented Dec 5, 2022

There is no such thing as "standard" markdown

I meant this as of OP sentence:

The current README file is not formatted for GitHub.

GitHub might be not the no. 1 priority, in which case a more generic Markdown syntax way be preferred. E.g. in the open PR

stick to adding extra whitespace

is not done. But otherwise it should be fine.

Generally as a "standard" Markdown, I mean basically this syntax, which should work as intended with every dialect interpreter:

the readme could be separate to a shiny part

Which means two files to maintain.

Don't understand me wrong, I don't want to argue against it and have no vote here anyway. Just want to tell OP why those "necessary changes to formatting" might not be seen as necessary by the maintainer. E.g. the same is true for a "shiny" Dropbear logo, I was wondering about 🙂: #196

@bkil
Copy link

bkil commented Dec 5, 2022

The idea to "separate" one big file to two smaller halves without intersection means that the overall maintenance burden should be the same.

Certain people mean "CommonMark" when they refer to standardized markdown I think:
https://en.wikipedia.org/wiki/Markdown#Standardization

@MichaIng
Copy link
Contributor

MichaIng commented Dec 5, 2022

The idea to "separate" one big file to two smaller halves without intersection means that the overall maintenance burden should be the same.

Ah, but since GitHub only shows one readme, whatever you add with the second file, will be missing?

Certain people mean "CommonMark"

Ah right, that is the most complete basic syntax description 👍.

@bkil
Copy link

bkil commented Dec 5, 2022

One can indeed hyperlink other documents from the welcome/landing page on GitHub. The other documents could be in a textual format even.

@MichaIng
Copy link
Contributor

MichaIng commented Dec 5, 2022

But those hyperlinked documents (unless images and such) cannot be expanded, is it? I know there are Markdown extensions which allow loading of external files, but I'm not aware of GitHub dialect supporting this.

@bkil
Copy link

bkil commented Dec 5, 2022

I don't know of a dialect that supported inlining either.

Some support various HTML tags within the Markdown code, but I consider that cheating.

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 a pull request may close this issue.

3 participants