Skip to content

Conversation

@felixfontein
Copy link
Collaborator

Addressing #49 (comment)

@felixfontein felixfontein changed the title Add RST syntax check [WIP] Add RST syntax check Jun 22, 2022
@briantist
Copy link
Collaborator

Alright looks like I got it; I think warning is the right level of things to fail on, because if we let warnings through, it would fail for the strictest settings in the rest of the build.

And the original thing about the links not being referenced was info level so I think this works out perfectly?
Will let you confirm when you've got time.

@felixfontein felixfontein changed the title [WIP] Add RST syntax check Add RST syntax check Jun 22, 2022
@felixfontein
Copy link
Collaborator Author

Feel free to merge :)

@briantist briantist merged commit 1ed81cb into ansible-community:main Jun 23, 2022
@briantist
Copy link
Collaborator

Thanks for your continued work on this @felixfontein !

@felixfontein felixfontein deleted the ci branch June 23, 2022 06:00
@felixfontein
Copy link
Collaborator Author

Thanks for polishing and merging this!

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.

2 participants