Format README #408
No reviewers
Labels
No Label
has
backport
in progress
invalid
kind
breaking
kind
bug
kind
build
kind
dependency
kind
deployment
kind
docs
kind
enhancement
kind
feature
kind
lint
kind
proposal
kind
question
kind
refactor
kind
security
kind
testing
kind
translation
kind
ui
need
backport
priority
critical
priority
low
priority
maybe
priority
medium
reviewed
duplicate
reviewed
invalid
reviewed
wontfix
skip-changelog
status
blocked
status
needs-feedback
status
needs-reviews
status
wip
upstream
gitea
upstream
other
No Milestone
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: gitea/helm-chart#408
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "format-readme"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Some small formatting changes which are auto-applied by the markdown code styler in vscode (which I am using a lot).
These are also present in #407.
Merging this before/in general would help be a bit when tinkering with the README.
Mainly indentation related, plus some single quotes -> double quotes changes.
Everything that can be covered by Markdownlint should be configured there. Otherwise, changing the README will be a mess when using different IDEs.
https://github.com/DavidAnson/markdownlint/blob/main/doc/Rules.md
If we commit on using such VSCode plugin, it can be mentioned within CONTRIBUTING.md along with the already recommended VSCode plugins.
I haven't found a way in markdownlint to address the multiline-quote thingy you see in the diff of this PR.
Same for the double quotes/single quotes change (even though I think this must be configurable somehow).
WRT to the indentation changes: I think the current README format is off for all the changes proposed in the PR, e.g. the
mysql:
andpostgresql:
keys are top-level invaluyes.yaml
and should not have indentation in the README.To make things more strict, I also added an
.editorconfig
file even though this does not have direct influence on the README changes here.One things is certainly to catch such offsets in CI, the other thing is to get README.md into a "proper" state for the moment :)
Adding the editorconfig ist a great idea.
I've just seen the "tab for every file" as indent. This won't work for yaml files. We should use spaces in editorconfig instead.
Good catch, that was a mistake by my side, I am also in favor of spaces.
@pat-s Which Vscode plugin do you use that formatted the markdown file? It can be mentioned within CONTRIBUTING.md along with the already recommended VSCode plugins.
Otherwise lgtm
I am using https://marketplace.visualstudio.com/items?itemName=esbenp.prettier-vscode but there are tons of markdown formatters out there.
But yeah let's add it to the list in CONTRIBUTING.md!