Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(copybara): sync commits from Aspect-internal silo #329

Merged
merged 10 commits into from
Nov 14, 2022

Conversation

gregmagolan and others added 10 commits November 14, 2022 10:30
4.2.2 just adds the coverage command which was missing. We don't use
that in silo but cut a release to exercise the release process in silo &
update commit followed by copybara & a release in OSS

GitOrigin-RevId: f3ca1f717a684877b857a604fd622137384ef69a
Fixes #753.

Signed-off-by: Thulio Ferraz Assis <3149049+f0rmiga@users.noreply.github.com>
GitOrigin-RevId: 028ee44620aae0f6b0f9e184a06639a4827066ea
gazelle seems to treat the //release package differently 🤷

GitOrigin-RevId: 62b17cc8208d71e60ee80df0e5947f2e7d7cacad
…no]workspace_config and --aspect:[no]home_config flags (#761)

Config file is now loaded before cobra is configured so that we can
check for a `version` config value in a follow-up.

Configs are loaded in increasing preference. Options in later files can
override a value form an earlier file if a conflict arises. Inspired by
where Bazel looks for .bazelrc and how this is configured:
https://bazel.build/run/bazelrc. I did not add a "system" config
equivalent to the "system" bazelrc found at `/etc/bazel.bazelrc`. It
could be easily added in the future with the pattern in place here.

GitOrigin-RevId: b412d765d3a5a66cfe7c26247d305af001725d72
Turns out that viper does not do the right thing unless you set both
`SetConfigName` without the extension and `SetConfigType` with extension
🤷

GitOrigin-RevId: a515b15a0c4126327baaa516b2ef1eeab2f48bca
Short-circuit the `--version` and `-v` special cases before we download
plugins so it is fast and doesn't require any network access

GitOrigin-RevId: aaf8a199878d6d28ed5a2d3ca16f868ea70dc9ea
…lags that expand to other flags (#762)

Issue reported by Matt here
https://aspect-build.slack.com/archives/C024P0WT7D4/p1668187960666339

GitOrigin-RevId: 5662ddf1599352bbc3ac1a4d7f3e875339ddf0b7
…f fetching aspect CLI (#765)

GitOrigin-RevId: 0e2b5e815e3eb05eb9ccd7dd7c77d16b7d8b3e29
@gregmagolan gregmagolan merged commit 7a7d9fb into main Nov 14, 2022
@gregmagolan gregmagolan deleted the 9E6ED7960E7C1C7500F98C1F23B6018B branch November 14, 2022 19:29
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.

2 participants