Skip to content

feat: Project list, get and set commands#22

Merged
nterbogt merged 11 commits into
mainfrom
project-get-set
Apr 2, 2026
Merged

feat: Project list, get and set commands#22
nterbogt merged 11 commits into
mainfrom
project-get-set

Conversation

@nterbogt
Copy link
Copy Markdown
Contributor

@nterbogt nterbogt commented Oct 12, 2025

  • skpr project list
  • skpr project get
  • skpr project set contact "me@example.com"

@nterbogt nterbogt force-pushed the project-get-set branch 2 times, most recently from e208733 to fab99b0 Compare November 3, 2025 19:34
@nterbogt nterbogt changed the title Initial project get and set feat: Project list, get and set commands Apr 1, 2026
@nterbogt nterbogt requested a review from Copilot April 1, 2026 21:39
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds new skpr project CLI capabilities to list projects, get current project details, and set project attributes (contact/tags).

Changes:

  • Introduces skpr project list, skpr project get, and skpr project set <key> <value> commands wired into the root CLI.
  • Adds project environment formatting helper (ListEnvironmentsByName) plus a unit test.
  • Implements list/get table output and set operations via the API client.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 9 comments.

Show a summary per file
File Description
internal/project/project.go Adds helper to order environments for display (currently mutates input slice).
internal/project/project_test.go Tests environment ordering helper.
internal/command/project/set/command.go Implements setting project attributes (contact, tags).
internal/command/project/list/command.go Implements listing projects and rendering a table (has package/comment/copy-paste issues).
internal/command/project/get/command.go Implements fetching and printing current project details (comment mismatch).
cmd/skpr/project/set/command.go Adds Cobra wiring for skpr project set.
cmd/skpr/project/list/command.go Adds Cobra wiring for skpr project list.
cmd/skpr/project/get/command.go Adds Cobra wiring for skpr project get (missing args validation; help text mismatch).
cmd/skpr/project/command.go Registers project command and subcommands (comment mismatch).
cmd/skpr/main.go Registers the new top-level project command.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread internal/command/project/list/command.go Outdated
Comment thread internal/command/project/list/command.go Outdated
Comment thread internal/command/project/list/command.go Outdated
Comment thread internal/project/project.go Outdated
Comment thread internal/command/project/get/command.go Outdated
Comment thread internal/command/project/set/command.go Outdated
Comment thread cmd/skpr/project/get/command.go Outdated
Comment thread cmd/skpr/project/command.go Outdated
Comment thread cmd/skpr/project/info/command.go
nterbogt and others added 8 commits April 2, 2026 10:54
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@nterbogt
Copy link
Copy Markdown
Contributor Author

nterbogt commented Apr 1, 2026

Might be worth considering a rename of the get command to info to match other interfaces in the CLI.

@nterbogt nterbogt merged commit 5447bbd into main Apr 2, 2026
2 checks passed
@nterbogt nterbogt deleted the project-get-set branch April 2, 2026 03:32
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.

3 participants