Update go to 1.17, codegangsta/cli to urfave/cli - Githubissues

cyberark / summon

CLI that provides on-demand secrets access for common DevOps tools

https://cyberark.github.io/summon

MIT License

705 stars 63 forks source link

Update go to 1.17, codegangsta/cli to urfave/cli #226

Closed andytinkham closed 2 years ago

andytinkham commented 2 years ago

Signed-off-by: Andy Tinkham andy.tinkham@cyberark.com

Desired Outcome

Update go to 1.17

Implemented Changes

Upgrades go to 1.17
Converts github.com/codegangsta/cli to its new url: github.com/urfave/cli
go mod tidy

Definition of Done

At least 1 todo must be completed in the sections below for the PR to be merged.

Changelog

[X] The CHANGELOG has been updated, or
[ ] This PR does not include user-facing changes and doesn't require a CHANGELOG update

Test coverage

[ ] This PR includes new unit and integration tests to go with the code changes, or
[X] The changes in this PR do not require tests

Documentation

[ ] Docs (e.g. READMEs) were updated in this PR
[ ] A follow-up issue to update official docs has been filed here: [insert issue ID]()
[X] This PR does not require updating any documentation

Behavior

[ ] This PR changes product behavior and has been reviewed by a PO, or
[ ] These changes are part of a larger initiative that will be reviewed later, or
[X] No behavior was changed with this PR

Security

[X] Security architect has reviewed the changes in this PR,
[ ] These changes are part of a larger initiative with a separate security review, or
[ ] There are no security aspects to these changes