-
Notifications
You must be signed in to change notification settings - Fork 1
docs: improve readme #24
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
Conversation
We should merge #23 first. |
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updates.
Could you fix points the linter pointed out?
Yep! I won't make the mistake of ignoring the linter again 😆 |
This PR is currently BLOCKED by the website link being a 404. This PR should not be merged until the website exists. |
I've added some MarkdownLint disable comments: <!-- markdownlint-disable MD041 MD033 --> Why?
|
Note that only certain tags can be allowed. |
@jcbhmr Could you take a look at this? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you like to review the Linter settings as I commented above?
@jcbhmr Do you still plan to work on this project? |
closes #6
This doesn't remove the under construction banner (since we don't have anything to show for this repo yet), but it does add some badly needed content to the readme!