Skip to content

do not ignore global flags2#162

Closed
skevetter wants to merge 3 commits intomainfrom
do-not-ignore-global-flags2
Closed

do not ignore global flags2#162
skevetter wants to merge 3 commits intomainfrom
do-not-ignore-global-flags2

Conversation

@skevetter
Copy link
Owner

  • Currently, some of the DevPod CLI sub-commands do not honor global flags --context and --provider.
  • fixup!: add context config loader
  • fixup!: add tests

dubinsky and others added 3 commits December 25, 2025 21:00
…ags `--context` and `--provider`.

For instance:
```shell
$ devpod context create box
$ devpod ide use intellij --context default
```
sets the default ide on the 'box' context and not on the 'default' context, as can be verified with

```shell
$ devpod ide list
$ devpod context use default
$ devpod ide list
```
which is wrong ;)

The fix seems to be to move a '*' from the structure creation to the structure declaration, the way most of the sub-commands do it.

I have only a vague idea of what this all means, this being the first Go code I ever touched, but it does bring up C(++) memories from the last millennium...

I applied this adjustment to all the 13 commands that seem to have misplaced that crucial '*':
- context create/delete/list/set-options/use
- ide list/options/set-options/use
- provider list/list-available/set-options/use

It is not clear to me how this was not detected earlier ;)

Please review!

Signed-off-by: Leonid Dubinsky <dub@podval.org>
Signed-off-by: Samuel K <69881238+skevetter@users.noreply.github.com>
Signed-off-by: Samuel K <69881238+skevetter@users.noreply.github.com>
@skevetter skevetter closed this Dec 26, 2025
@skevetter skevetter deleted the do-not-ignore-global-flags2 branch February 12, 2026 17:23
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