-
Notifications
You must be signed in to change notification settings - Fork 806
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
feat: Test if all providers are documented #2034
feat: Test if all providers are documented #2034
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎ 1 Skipped Deployment
|
…iders-should-be-documented' into Matvey-Kuk/all-providers-should-be-documented
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 separate the new docs thingy in to a different workflow that only triggers when providers/** directory changes. It makes sense not to run it every time and in general I’m paranoid about all these scripts :( (bad past experience 😅)
@talboren good idea, moved! |
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.
LGTM
Noticed that ServiceNow provider is not documented or mentioned in our docs. This test will fail if one will add a provider without docs.
Closes #2035