Skip to content

Fix duplicate "apps" entry in help output - #2191

Merged
pietern merged 1 commit into
mainfrom
fix-double-apps
Jan 20, 2025
Merged

Fix duplicate "apps" entry in help output#2191
pietern merged 1 commit into
mainfrom
fix-double-apps

Conversation

@pietern

Copy link
Copy Markdown
Contributor

Changes

This is not needed because the command group is already returned by workspace.All().

The additional command registration was added in #1679.

Tests

Acceptance test.

@pietern
pietern added this pull request to the merge queueJan 20, 2025
Merged via the queue into main with commit 69f3c0aJan 20, 2025
@pietern
pietern deleted the fix-double-apps branch January 20, 2025 16:10
andrewnester added a commit that referenced this pull request Jan 23, 2025
CLI:
* Added text output templates for apps list and list-deployments ([#2175](#2175)).
* Fix duplicate "apps" entry in help output ([#2191](#2191)).
Bundles:
* Allow yaml-anchors in schema ([#2200](#2200)).
* Show an error when non-yaml files used in include section ([#2201](#2201)).
* Set WorktreeRoot to sync root outside git repo ([#2197](#2197)).
* fix: Detailed message for using source-linked deployment with file_path specified ([#2119](#2119)).
* Allow using variables in enum fields ([#2199](#2199)).
* Add experimental-jobs-as-code template ([#2177](#2177)).
* Reading variables from file ([#2171](#2171)).
* Fixed an apps message order and added output test ([#2174](#2174)).
* Default to forward slash-separated paths for path translation ([#2145](#2145)).
* Include a materialized copy of built-in templates ([#2146](#2146)).
github-merge-queueBot pushed a commit that referenced this pull request Jan 23, 2025
CLI:
* Added text output templates for apps list and list-deployments
([#2175](#2175)).
* Fix duplicate "apps" entry in help output
([#2191](#2191)).
Bundles:
* Allow yaml-anchors in schema
([#2200](#2200)).
* Show an error when non-yaml files used in include section
([#2201](#2201)).
* Set WorktreeRoot to sync root outside git repo
([#2197](#2197)).
* fix: Detailed message for using source-linked deployment with
file_path specified
([#2119](#2119)).
* Allow using variables in enum fields
([#2199](#2199)).
* Add experimental-jobs-as-code template
([#2177](#2177)).
* Reading variables from file
([#2171](#2171)).
* Fixed an apps message order and added output test
([#2174](#2174)).
* Default to forward slash-separated paths for path translation
([#2145](#2145)).
* Include a materialized copy of built-in templates
([#2146](#2146)).
denik pushed a commit that referenced this pull request May 20, 2026
## Changes
This is not needed because the command group is already returned by
`workspace.All()`.
The additional command registration was added in #1679.
## Tests
Acceptance test.
denik pushed a commit that referenced this pull request May 20, 2026
CLI:
* Added text output templates for apps list and list-deployments
([#2175](#2175)).
* Fix duplicate "apps" entry in help output
([#2191](#2191)).
Bundles:
* Allow yaml-anchors in schema
([#2200](#2200)).
* Show an error when non-yaml files used in include section
([#2201](#2201)).
* Set WorktreeRoot to sync root outside git repo
([#2197](#2197)).
* fix: Detailed message for using source-linked deployment with
file_path specified
([#2119](#2119)).
* Allow using variables in enum fields
([#2199](#2199)).
* Add experimental-jobs-as-code template
([#2177](#2177)).
* Reading variables from file
([#2171](#2171)).
* Fixed an apps message order and added output test
([#2174](#2174)).
* Default to forward slash-separated paths for path translation
([#2145](#2145)).
* Include a materialized copy of built-in templates
([#2146](#2146)).
Sign up for freeto 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

@pietern@denik