Add split Linux docs jobs to prod builder json #78888
Conversation
jmagman
added
the
c: contributor-productivity
jmagman
changed the title
| "task_name": "linux_docs_test", | ||
| "flaky": true | ||
| }, | ||
| { |
There was a problem hiding this comment.
Should Linux docs_publish run only in release branches? adding it here will run it for master.
There was a problem hiding this comment.
I think it's supposed to run on master, just not the release candidate branches:
https://github.com/flutter/infra/blob/e83d25832f6b56a7651365076eb842f57328343a/config/packaging_config.star#L149-L152
# Docs builds are only done on master and stable channels
if branch in ("master", "stable"):
common.linux_prod_builder(
name = "Linux%s docs_publish|docs" % ("" if branch == "master" else " " + branch),
I'll wait until @christopherfujino comes back from vacation, he can validate that's what he intended.
There was a problem hiding this comment.
this change is correct, provided we follow-up to mark un-flaky.
confusingly, docs are published only on master (https://master-api.flutter.dev) and stable channels (https://api.flutter.dev).
3 participants
The
Linux docsprod builder was split intoLinux docs_testandLinux docs_publishin https://github.com/flutter/infra/pull/375. Add them to the framework builders json file.Linux docswas showing up as not running:Note the try job is still called
Linux docs.Fixes #78887