[update] change Error: to Warning: for failed attempt to upgrade flake

[savil]

Summary

When I modified the examples/flakes/php to omit ds extension and did devbox update, then the output alarmed me at first.

The Error made me think something drastic went wrong. But in this case, since its an "attempt", I think we should
make it a Warning or even an Info.

❯ devbox update
Info: Attempting to upgrade path:my-php-flake#php using `nix profile upgrade`
Error: Failed to upgrade path:my-php-flake#php using `nix profile upgrade`: php: package not found
Info: Attempting to upgrade path:my-php-flake#hello using `nix profile upgrade`
Error: Failed to upgrade path:my-php-flake#hello using `nix profile upgrade`: hello: package not found
Info: Already up-to-date ripgrep@latest 14.1.0
[1/2] php
[1/2] php: Success
[2/2] hello
[2/2] hello: Success
✓ Computed the Devbox environment.
Info: Removing [email protected]
Warning: Your shell environment may be out of date. Run `refresh` to update it.
Info: Running "nix flake update"
(devbox)

How was it tested?

❯ devbox update
Info: Attempting to upgrade path:my-php-flake#php using `nix profile upgrade`
Warning: Failed to upgrade path:my-php-flake#php using `nix profile upgrade`: php: package not found
Info: Attempting to upgrade path:my-php-flake#hello using `nix profile upgrade`
Warning: Failed to upgrade path:my-php-flake#hello using `nix profile upgrade`: hello: package not found
Info: Already up-to-date ripgrep@latest 14.1.0
[1/2] php
[1/2] php: Success
[2/2] hello
[2/2] hello: Success
✓ Computed the Devbox environment.
Info: Removing [email protected]
Warning: Your shell environment may be out of date. Run `refresh` to update it.
Info: Running "nix flake update"
(devbox)

[savil]

Current dependencies on/for this PR:

• main
  • PR [update] change Error: to Warning: for failed attempt to upgrade flake #1773 👈

This stack of pull requests is managed by Graphite.

[Lagoja]

Why does the warning show at all if you're running devbox update?

[savil]

Why does the warning show at all if you're running devbox update?

its being transparent about what specifically we're trying to update and if it worked or didn't?

[mikeland73]

Did we decide this can be removed? We need to ensure nix build creates the latest version of a flake (I assume it does) in which case we no longer need to upgrade flakes.

[mikeland73]

.

[savil]

@mikeland73 lets not conflate two different discussions. I need to investigate whether this code is still needed. That will take some time given my other commitments. In the meantime, we should fix this misleading "Error:" message.