Skip to content

feat(internal/command): add command execution package with logging#2968

Merged
julieqiu merged 2 commits intogoogleapis:mainfrom
julieqiu:cmd
Nov 20, 2025
Merged

feat(internal/command): add command execution package with logging#2968
julieqiu merged 2 commits intogoogleapis:mainfrom
julieqiu:cmd

Conversation

@julieqiu
Copy link
Copy Markdown
Member

An internal/command package is created to execute external commands and print them to stderr before execution, in order to provide progress reports to the user.

Existing call sites from internal/sidekick/external are migrated to use the new package and the old package is deleted.

For #2966

@julieqiu julieqiu requested a review from coryan November 20, 2025 02:27
@julieqiu julieqiu marked this pull request as ready for review November 20, 2025 02:27
@julieqiu julieqiu requested a review from a team November 20, 2025 02:28
@julieqiu julieqiu requested a review from a team as a code owner November 20, 2025 02:28
@julieqiu julieqiu removed request for a team November 20, 2025 02:28
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 20, 2025

Codecov Report

❌ Patch coverage is 34.37500% with 21 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.12%. Comparing base (005f697) to head (cff526e).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
internal/sidekick/sidekick/rust_generate.go 0.00% 13 Missing ⚠️
internal/sidekick/rust_prost/generate.go 0.00% 6 Missing ⚠️
internal/sidekick/dart/dart.go 0.00% 1 Missing ⚠️
internal/sidekick/rust_release/bump_versions.go 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2968      +/-   ##
==========================================
- Coverage   83.15%   83.12%   -0.03%     
==========================================
  Files         137      137              
  Lines       11053    11052       -1     
==========================================
- Hits         9191     9187       -4     
- Misses       1445     1448       +3     
  Partials      417      417              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@julieqiu julieqiu requested review from a team and removed request for coryan November 20, 2025 02:48
@julieqiu julieqiu enabled auto-merge (squash) November 20, 2025 02:50
dbolduc
dbolduc previously approved these changes Nov 20, 2025
An internal/command package is created to execute external commands and
print them to stderr before execution, in order to provide progress
reports to the user.

Existing call sites from internal/sidekick/external are migrated to use
the new package and the old package is deleted.
@julieqiu julieqiu merged commit 12f4e94 into googleapis:main Nov 20, 2025
9 checks passed
ldetmer added a commit that referenced this pull request Nov 20, 2025
PR created by the Librarian CLI to initialize a release. Merging this PR
will auto trigger a release.

Librarian Version: not available
Language Image:
us-central1-docker.pkg.dev/cloud-sdk-librarian-prod/images-prod/librarian-release-container:latest
<details><summary>librarian: 0.7.0</summary>

##
[0.7.0](v0.6.0...v0.7.0)
(2025-11-20)

### Features

* add command execution package with logging (#2968)
([12f4e94](12f4e949))

* add branch support to RepositoryConfig (#2906)
([2cab5d6](2cab5d65))

* add image sha as a config in repositories config (#2954)
([3d1f537](3d1f5375))

* add initial config (#2965)
([502b063](502b063c))

* add fetch package (#2964)
([53de44d](53de44d8))

* parse gapic_metadata for API version release notes (#2815)
([7ee0328](7ee03283))

* onboard 25 repositories to automation for publish-release (#2907)
([87fae04](87fae04a))

* add version command (#2894)
([8f85be0](8f85be09))

* add RequireCommand test helper (#2978)
([a820a71](a820a710))

* add initial command (#2961)
([c959aea](c959aeae))

* add `stage-release` command (#2902)
([ee05006](ee050065))

### Bug Fixes

* require git v2.28.0 (#2963)
([265ee47](265ee47f))

* enums referenced outside the package (#2952)
([7331816](73318169))

</details>
@julieqiu julieqiu deleted the cmd branch December 31, 2025 00:28
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