Skip to content

Enable experimental_tarball_format for container_bundle. - #1938

Merged
alexeagle merged 1 commit into
bazelbuild:masterfrom
aptenodytes-forsteri:experimental-tarball-format-for-container-bundle
Oct 11, 2021
Merged

Enable experimental_tarball_format for container_bundle.#1938
alexeagle merged 1 commit into
bazelbuild:masterfrom
aptenodytes-forsteri:experimental-tarball-format-for-container-bundle

Conversation

@aptenodytes-forsteri

Copy link
Copy Markdown
Contributor

Resolves TODO. Seems to improve build time.

PR Checklist

Please check if your PR fulfills the following requirements:

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Code style update (formatting, local variables)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • CI related changes
  • Documentation content changes
  • Other... Please describe:

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@aptenodytes-forsteri

Copy link
Copy Markdown
Contributor Author

Looks like I need to update docs.

Resolves TODO. Seems to improve build time.
@aptenodytes-forsteri
aptenodytes-forsteri force-pushed the experimental-tarball-format-for-container-bundle branch from b2fc358 to 4ef6b39 Compare October 11, 2021 04:16
@aptenodytes-forsteri

Copy link
Copy Markdown
Contributor Author

Ran bazel run //docs:update per instructions here:

# If these fail, run `bazel run //docs:update`

@alexeagle

Copy link
Copy Markdown
Contributor

it's the next step in #1695 (comment) IIUC

@alexeagle
alexeagle merged commit af2b074 into bazelbuild:master Oct 11, 2021
@alexeagle

Copy link
Copy Markdown
Contributor

Thanks!

@aptenodytes-forsteri
aptenodytes-forsteri deleted the experimental-tarball-format-for-container-bundle branch October 13, 2021 02:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants