Feat/easy install#4
Conversation
- Add install.sh for one-command setup - Make scripts derive KCTL_ENV_ROOT from install path - Fix list-remote via GitHub tags API - Document quick install in README
Replace <kctl-env@example.com> with sen.net@hotmail.com in packaging metadata and release script.
There was a problem hiding this comment.
Pull request overview
Adds a bootstrap installer and updates remote version discovery to keep kctl-env easy to install/use as upstream distribution endpoints evolve.
Changes:
- Added
install.shbootstrap installer and documented “Quick install” flow in README. - Updated
kctl-env list-remoteto use the GitHub tags API instead of parsingdl.k8s.io. - Adjusted
KCTL_ENV_ROOTdefaulting logic in shims and updated packaging/release maintainer metadata.
Reviewed changes
Copilot reviewed 8 out of 10 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| scripts/release.sh | Updates release automation output (Deb/RPM changelog author). |
| packaging/rpm/kctl-env.spec | Changes RPM changelog maintainer email. |
| packaging/debian/control | Changes Debian maintainer email. |
| packaging/debian/changelog | Changes Debian changelog signature email. |
| libexec/kctl-env-list-remote | Switches remote version listing to GitHub tags API. |
| install.sh | Adds a zero-deps bootstrap installer for kctl-env. |
| bin/kubectl | Changes default KCTL_ENV_ROOT resolution to be path-relative. |
| bin/kctl-env | Changes default KCTL_ENV_ROOT resolution to be path-relative. |
| README.md | Documents quick-install via `curl |
| CHANGELOG.md | Adds an Unreleased section noting the installer/docs changes. |
Comments suppressed due to low confidence (2)
packaging/debian/changelog:6
- This changelog signature switches to a personal Hotmail address. For Debian changelogs, the signature should typically reflect a stable maintainer/project contact (and match the control file) to avoid long-term maintenance and provenance issues.
-- kctl-env Maintainers <sen.net@hotmail.com> Wed, 07 Jan 2026 22:00:00 +0000
packaging/rpm/kctl-env.spec:41
- RPM spec changelog entry now uses a personal Hotmail address. For package metadata, it's generally better to use a project/team contact address (or distro maintainer) rather than an individual's mailbox, to keep provenance/support stable over time.
%changelog
* Wed Jan 07 2026 kctl-env Maintainers <sen.net@hotmail.com> - 0.1.0-1
- Initial release
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot open a new pull request to apply changes based on the comments in this thread |
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
|
@copilot open a new pull request to apply changes based on the comments in this thread |
…ss (#6) * Initial plan * fix: add curl -f flag and revert maintainer emails to project address Co-authored-by: senet <4061835+senet@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: senet <4061835+senet@users.noreply.github.com>
|
@copilot open a new pull request to apply changes based on the comments in this thread |
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
|
@copilot review and approve |
Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 8 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Prosenjit Sen <4061835+senet@users.noreply.github.com>
* Initial plan * Add macOS compatibility for SHA256 verification with fallback support Co-authored-by: senet <4061835+senet@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: senet <4061835+senet@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 8 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Prosenjit Sen <4061835+senet@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Prosenjit Sen <4061835+senet@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Prosenjit Sen <4061835+senet@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Prosenjit Sen <4061835+senet@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Prosenjit Sen <4061835+senet@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Prosenjit Sen <4061835+senet@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Prosenjit Sen <4061835+senet@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 8 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Prosenjit Sen <4061835+senet@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Prosenjit Sen <4061835+senet@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Prosenjit Sen <4061835+senet@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Prosenjit Sen <4061835+senet@users.noreply.github.com>
Read only first hash from checksum file, normalize case, validate SHA256, and require tr.
- Avoid referencing old tags without install.sh - Document immutable-release checksum attachment
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 6 out of 8 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Signed-off-by: Prosenjit Sen <4061835+senet@users.noreply.github.com>
Summary
Changes
Checklist
mainmainREADME.md/ docs if neededCHANGELOG.mdfor user-facing changesmake release V=X.Y.Zand opened PR fromrelease/vX.Y.Z