You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Right now we change the content, merge it, and hope the build will be successful 🤞
We have tried to add some frontmatter validation, but it is hard to know the strange issues that will cause build failures, e.g. whitespace issue we had faced recently #1448
💡 Suggested Solution
New required PR check - on content repo Pull Requests try to build changed files (this will catch all the issues and future proof the solution)
Tasks
Add new PR check as per description in the above
For week leave it not required, and see if it is working.
Then make it required check after successful tests
Thanks!
The text was updated successfully, but these errors were encountered:
Cc: @adamcogan @bradystroud @tkapa @Calinator444 @tiagov8
Hi team
❌ Pain
Right now we change the content, merge it, and hope the build will be successful 🤞
We have tried to add some frontmatter validation, but it is hard to know the strange issues that will cause build failures, e.g. whitespace issue we had faced recently #1448
💡 Suggested Solution
New required PR check - on content repo Pull Requests try to build changed files (this will catch all the issues and future proof the solution)
Tasks
Thanks!
The text was updated successfully, but these errors were encountered: