Skip to content
Snippets Groups Projects

cli-auth-config: explicitly message about skipped token update

Merged Pierre-Yves David requested to merge marmoute/swh-auth:explicit-skip into master
1 unresolved thread

At the end of a auth-config command run, we ask the user if the newly obtained token should be written on disk or discarded.

The default is to discard it (which seems a bit weird, but is not the point here). Unlike what we do on write, where a green confirmation message is displayed, we do not display any message when discarding. So a user doing things a bit too fast can imagine that the update went all fine as it just display a wall of green messages.

So we add a simple yellow message pointing out that we ultimately did not write the new token. That should fix the problem.

Merge request reports

Pipeline #8605 passed

Pipeline passed for 11734334 on marmoute:explicit-skip

Approval is optional

Merged by Antoine LambertAntoine Lambert 10 months ago (May 14, 2024 8:21am UTC)

Merge details

  • Changes merged into master with 11734334.
  • Deleted the source branch.

Pipeline #8705 passed

Pipeline passed for 11734334 on master

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading