Skip to content

Conversation

@StaticRocket
Copy link
Member

Red Hat has a pretty good template for this thing [1] and it already understands RST. Might as well pick it up here. Fair warning, it's much more picky than I am. I'm not entirely sure if we should add it to the PR checker right now, as even my stuff was flagged to hell and back.

[1] https://github.com/redhat-documentation/vale-at-red-hat

May be good enough to address #19 in the future though.

Red Hat has a pretty good template for this thing [1] and it already
understands RST. Might as well pick it up here. Fair warning, it's much
more picky than I am. I'm not entirely sure if we should add it to the
PR checker right now, as even my stuff was flagged to hell and back.

[1] https://github.com/redhat-documentation/vale-at-red-hat

Signed-off-by: Randolph Sapp <[email protected]>
@StaticRocket
Copy link
Member Author

I do want people to test this out and give their opinion before we go down this road. It'll be a lot of work to match Red Hat's standards for documentation.

@cshilwant
Copy link
Member

I do want people to test this out and give their opinion before we go down this road. It'll be a lot of work to match Red Hat's standards for documentation.

Adding @jeevantelukula @sadik-smd @jsuhaas22 for reviews

@nmenon
Copy link
Contributor

nmenon commented May 16, 2025

This looks good. lets get this merged. and add to pr checker

@cshilwant cshilwant merged commit 35929da into TexasInstruments:master May 16, 2025
2 checks 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.

5 participants