-
Notifications
You must be signed in to change notification settings - Fork 4
chore: add CI step for validating contributor READMEs #1
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
# Coder | ||
|
||
Coder provisions cloud development environments via Terraform, supporting Linux, macOS, Windows, X86, ARM, Kubernetes and more. |
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.
This hasn't come up in any of the conversations yet, but I think that it's safe to give developers the freedom to throw whatever they want in the body. We'll just make sure the code always ignores it
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.
Mainly just code styling changes, but overall this looks like a great start for validation.
registry/nataindata/README.md
Outdated
--- | ||
display_name: Nataindata | ||
bio: Data engineer | ||
github: nataindata | ||
website: https://www.nataindata.com | ||
status: partner | ||
--- |
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.
If it's not a lot of work, could you drop a screenshot or mock of what the page should looks like with just the bare minimum frontmatter filled out?
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.
Are they a partner? Shouldn't this be like community
or something?
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.
Yeah, updated the field, and I'll make a quick mockup tomorrow morning.
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.
Clean!
Closes coder/internal#530
Changes made
Notes