Skip to content

Commit

Permalink
pw_presubmit: Make gn_docs_build accessible
Browse files Browse the repository at this point in the history
Add the gn_docs_build step to the OTHER_CHECKS program so it can be
invoked with --step.

Also have it install nanopb.

Bug: b/285350808
Change-Id: Ib4689dbdbdcc2eed9f13a38dfa32aaf3b7531366
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/150133
Pigweed-Auto-Submit: Rob Mohr <[email protected]>
Reviewed-by: Anthony DiGirolamo <[email protected]>
Commit-Queue: Rob Mohr <[email protected]>
  • Loading branch information
mohrr authored and CQ Bot Account committed Jun 2, 2023
1 parent 98f2c3b commit 09129c4
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion pw_presubmit/py/pw_presubmit/pigweed_presubmit.py
Original file line number Diff line number Diff line change
Expand Up @@ -442,7 +442,9 @@ def gn_arm_build(ctx: PresubmitContext):
),
)

gn_docs_build = build.GnGenNinja(name='gn_docs_build', ninja_targets=('docs',))
gn_docs_build = build.GnGenNinja(
name='gn_docs_build', packages=('nanopb',), ninja_targets=('docs',)
)

gn_host_tools = build.GnGenNinja(
name='gn_host_tools',
Expand Down Expand Up @@ -1024,6 +1026,7 @@ def owners_lint_checks(ctx: PresubmitContext):
gitmodules.create(gitmodules.Config(allow_submodules=False)),
gn_clang_build,
gn_combined_build_check,
gn_docs_build,
module_owners.presubmit_check(),
npm_presubmit.npm_test,
pw_transfer_integration_test,
Expand Down

0 comments on commit 09129c4

Please sign in to comment.