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

[tmpnet] Rename tmpnetctl main package from cmd to tmpnetctl #3787

Merged
merged 1 commit into from
Mar 13, 2025

Conversation

maru-ava
Copy link
Contributor

@maru-ava maru-ava commented Mar 11, 2025

Why this should be merged

Previously the main package for tmpnetctl was named cmd, which meant the binary resulting from go install was cmd. Renaming the package to tmpnetctl is intended to allow downstream repos to go install tmptnetct for use in CI.

This is intended as a stopgap until tmpnetctl can be distributed with nix and avoid having to maintain a tmpnetctl script in downstream repos.

How this works

  • renames tmpnetctl main package
  • adds documentation for downstream use
  • updates use of tmpnetctl in the run-monitored-tmpnet-cmd custom action

How this was tested

CI

Need to be documented in RELEASES.md?

N/A

@maru-ava maru-ava added testing This primarily focuses on testing ci This focuses on changes to the CI process labels Mar 11, 2025
@maru-ava maru-ava self-assigned this Mar 11, 2025
@maru-ava maru-ava changed the title [tmpnet] Rename tmpnetctl package from cmd to tmpnetctl [tmpnet] Rename tmpnetctl main package from cmd to tmpnetctl Mar 11, 2025
@StephenButtolph StephenButtolph added this pull request to the merge queue Mar 13, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to a conflict with the base branch Mar 13, 2025
This enables downstream repos to `go install` `tmptnetctl` rather than `cmd`.
@maru-ava maru-ava force-pushed the tmpnet-rename-tmpnetctl-pkg branch from 6e41191 to a42b2a0 Compare March 13, 2025 18:32
@maru-ava maru-ava enabled auto-merge March 13, 2025 18:33
@maru-ava maru-ava added this pull request to the merge queue Mar 13, 2025
Merged via the queue into master with commit 664ee4b Mar 13, 2025
23 checks passed
@maru-ava maru-ava deleted the tmpnet-rename-tmpnetctl-pkg branch March 13, 2025 19:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci This focuses on changes to the CI process testing This primarily focuses on testing
Projects
Status: Done ✅
Development

Successfully merging this pull request may close these issues.

4 participants