Skip to content

#5421 no outdated dependencies#5437

Open
lupodevelop wants to merge 3 commits intogleam-lang:mainfrom
lupodevelop:#5421-no-outdated-dependencies
Open

#5421 no outdated dependencies#5437
lupodevelop wants to merge 3 commits intogleam-lang:mainfrom
lupodevelop:#5421-no-outdated-dependencies

Conversation

@lupodevelop
Copy link

@lupodevelop lupodevelop commented Mar 9, 2026

  • The changes in this PR have been discussed beforehand in an issue
  • The issue for this PR has been linked
  • Tests have been added for new behaviour
  • The changelog has been updated for any user-facing changes

#5421

This pull request improves the user experience of the gleam deps outdated command by making its output clearer when there are no outdated dependencies. Previously, the command was silent if no updates were found, which could be confusing. Now, it prints a helpful message in this case. The changes also include new tests to ensure this behavior.

Enhancements to gleam deps outdated output:

  • The command now prints No outdated dependencies found. when there are no outdated dependencies, instead of being silent. This is handled by introducing a new format_version_updates function.
  • Updated the CHANGELOG to document this improved behavior for users.

Always print a user-facing message for the gleam deps outdated command.
Copilot AI review requested due to automatic review settings March 9, 2026 12:06

This comment was marked as spam.

@lpil
Copy link
Member

lpil commented Mar 9, 2026

Hello! You've ticked the box saying this design has been discussed beforehand, but no such discussion has taken place.

@lpil
Copy link
Member

lpil commented Mar 9, 2026

Please do not run AI spam in our PRs, it is quite annoying!

@lupodevelop
Copy link
Author

lupodevelop commented Mar 9, 2026

Hello! You've ticked the box saying this design has been discussed beforehand, but no such discussion has taken place.

My fault. Wrong tick. "I looked too closely at the problem while I was on lunch break" until I wrote it down and sent the edit.

First time. First mistake 😅
Lesson learned.
If you think this fix is ​​not needed, feel free to discard it. no problem.

Please do not run AI spam in our PRs, it is quite annoying!

Are you talking about the copilot review? I didn't know it could be removed.

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