Skip to content

Docs: provide option to skip analysis in doc-check#282

Open
bkhouri wants to merge 1 commit into
swiftlang:mainfrom
bkhouri:t/main/gh281_add_doc_check_input_options
Open

Docs: provide option to skip analysis in doc-check#282
bkhouri wants to merge 1 commit into
swiftlang:mainfrom
bkhouri:t/main/gh281_add_doc_check_input_options

Conversation

@bkhouri

@bkhouri bkhouri commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Some projects may not want to run the --analyze option on the DocC generation until they are ready.

Provide an workflow input option to skip the analyze. While at it, update the check-docs.sh script to accept arugments instead of relying on environment variables being set. This allows more easily run the script at-desk.

Fixes #281

@bkhouri bkhouri force-pushed the t/main/gh281_add_doc_check_input_options branch 4 times, most recently from f31e5a2 to ebccca7 Compare June 10, 2026 14:23
@bkhouri

bkhouri commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

Since I am unable to re-trigger the failed GitHub Actions, I'm closing and re-opening the PR to trigger them

@bkhouri bkhouri closed this Jun 10, 2026
@bkhouri bkhouri reopened this Jun 10, 2026
@bkhouri bkhouri force-pushed the t/main/gh281_add_doc_check_input_options branch from ebccca7 to eae5369 Compare June 10, 2026 17:29
Comment thread docs/soundness-docs-check.md Outdated
@bkhouri bkhouri force-pushed the t/main/gh281_add_doc_check_input_options branch 2 times, most recently from 6e461f1 to ec74680 Compare June 10, 2026 17:46
@bkhouri bkhouri marked this pull request as ready for review June 10, 2026 17:48
@bkhouri bkhouri requested a review from a team as a code owner June 10, 2026 17:48
@bkhouri bkhouri force-pushed the t/main/gh281_add_doc_check_input_options branch 3 times, most recently from c9aa458 to 843dda3 Compare June 10, 2026 19:09
@bkhouri bkhouri marked this pull request as draft June 10, 2026 19:11
Some projects may not want to run the `--analyze` option on the DocC
generation until they are ready.

Provide an workflow input option to skip the analyze.  While at it,
update the `check-docs.sh` script to accept arugments instead of relying
on environment variables being set.  This allows more easily run the
script at-desk.

Fixes #2181
@bkhouri bkhouri force-pushed the t/main/gh281_add_doc_check_input_options branch 2 times, most recently from 2bfd438 to 07dc924 Compare June 10, 2026 19:30
@bkhouri bkhouri marked this pull request as ready for review June 10, 2026 20:19
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.

ADDITIONAL_DOCC_ARGUMENTS: unbound variable when running the Documentation check workflow

1 participant