You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Would become something like:
"New Services: 'networkmonitor'. Added 1 action for 'codecommit', 12 actions for 'networkmonitor', and 1 action for 'sagemaker'."
That issue from trackiam reminded me I've wanted to extend the readability/usefulness of the aws-iam-actions-list commit messages. I've wondered if it should even go as far as including all actions but that seems excessive (it is in the git commit/diff one click away after all). Either way, I think adding the count (total actions, added/removed, for that service) is a great idea...and I'll look to add that going forward. I wouldn't want to break people's forks but I would be willing to force change old commit messages to include that information retroactively too...not sure how that would work or if people would oppose it; that is what I did when I hard-forked the project that inspired this one to go from generic commit messages like "added actions" just like trackiam is currently doing.
The text was updated successfully, but these errors were encountered:
As suggested glassechidna/trackiam#7 ... to make the commit message more useful at a glance.
So, for example, the message from:
"New Services: 'networkmonitor'. Added actions to 'codecommit,networkmonitor,sagemaker'."
Would become something like:
"New Services: 'networkmonitor'. Added 1 action for 'codecommit', 12 actions for 'networkmonitor', and 1 action for 'sagemaker'."
That issue from trackiam reminded me I've wanted to extend the readability/usefulness of the aws-iam-actions-list commit messages. I've wondered if it should even go as far as including all actions but that seems excessive (it is in the git commit/diff one click away after all). Either way, I think adding the count (total actions, added/removed, for that service) is a great idea...and I'll look to add that going forward. I wouldn't want to break people's forks but I would be willing to force change old commit messages to include that information retroactively too...not sure how that would work or if people would oppose it; that is what I did when I hard-forked the project that inspired this one to go from generic commit messages like "added actions" just like trackiam is currently doing.
The text was updated successfully, but these errors were encountered: