Context:
From an internal Slack discussion: to simplify publishing HelpDot content, we want to exempt HelpDot articles from irrelevant engineering PR checks (GitHub Actions).
These checks should either auto-pass or be skipped (either one works) when a PR changes an affected file.
From @thienlnam
All of these are in App behind docs/articles and are markdown files so we should be able to update the tests to not run on these specific files
Added 8/27:
Let's get the checklist to stop pre-populating in the PR body too. Discussed here, as GitHub supports only one template for PRs we'll have to do a workaround to remove the checklist using a bot.
--
Files this will apply to
all .md files in docs/articles and sub-folders.
Affected tests
Skip or auto-pass these automated PR checks. Note that this is all automated PR checks except spell check:
- PR Reviewer Checklist / checklist (pull_request_review)
- imgbot Image Optimization / approveAndMerge (pull_request)
- PR Presubmit Reviews / review (pull_request)
- / Changed files ESLint check (pull_request)
- Check unused styles / Check for unused styles (pull_request)
- CLA Assistant / CLA / CLA (pull_request_target)
- ESLint check / ESLint check (pull_request)
- Generate static translations / dryRun (pull_request)
- Jest Unit Tests / Storybook tests (pull_request)
- Jest Unit Tests / test (job 1) (pull_request)
- Jest Unit Tests / test (job 2) (pull_request)
- Jest Unit Tests / test (job 3) (pull_request)
- license/snyk (expensify)
- PR Author Checklist / checklist (pull_request_target)
- Prettier check / Prettier check (pull_request)
- Reassure Performance Tests / perf-tests (pull_request)
- security/snyk (expensify)
- TypeScript Checks / typecheck (pull_request)
- Verify signed commits / verifySignedCommits (pull_request)
Issue Owner
Current Issue Owner: @stephanieelliott
Context:
From an internal Slack discussion: to simplify publishing HelpDot content, we want to exempt HelpDot articles from irrelevant engineering PR checks (GitHub Actions).
These checks should either auto-pass or be skipped (either one works) when a PR changes an affected file.
From @thienlnam
Added 8/27:
Let's get the checklist to stop pre-populating in the PR body too. Discussed here, as GitHub supports only one template for PRs we'll have to do a workaround to remove the checklist using a bot.
--
Files this will apply to
all .md files in docs/articles and sub-folders.
Affected tests
Skip or auto-pass these automated PR checks. Note that this is all automated PR checks except spell check:
Issue Owner
Current Issue Owner: @stephanieelliott