Skip to content

fix(cli): unhide storage and storage init commands.#1147

Merged
mergify[bot] merged 6 commits intoaws:masterfrom
bvtujo:storage-cmd-help
Jul 15, 2020
Merged

fix(cli): unhide storage and storage init commands.#1147
mergify[bot] merged 6 commits intoaws:masterfrom
bvtujo:storage-cmd-help

Conversation

@bvtujo
Copy link
Copy Markdown
Contributor

@bvtujo bvtujo commented Jul 15, 2020

This PR sets storage commands to visible and fixes a docstring.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@bvtujo bvtujo requested a review from a team as a code owner July 15, 2020 22:44
@bvtujo bvtujo requested a review from kohidave July 15, 2020 22:44
Copy link
Copy Markdown
Contributor

@iamhopaul123 iamhopaul123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚢

Comment thread go.sum
Comment on lines -419 to -420
github.com/spf13/afero v1.3.0 h1:Ysnmjh1Di8EaWaBv40CYR4IdaIsBc5996Gh1oZzCBKk=
github.com/spf13/afero v1.3.0/go.mod h1:5KUK8ByomD5Ti5Artl0RtHeI5pTF7MIDuXL3yY520V4=
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ugh why these don't go away automatically. Thanks for removing them.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm trying to get into the habit of running go mod tidy more often.

Comment thread go.mod Outdated
github.com/onsi/ginkgo v1.14.0
github.com/onsi/gomega v1.10.1
github.com/pelletier/go-toml v1.8.0 // indirect
github.com/pkg/errors v0.9.1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

huh do you know why this was added?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm honestly not sure; I ran go mod tidy and I don't quite know why this got picked up.

...

On further investigation it's added in the task pkg; looks like it got auto-added by goimports in #1145 when just "errors" would have worked.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ooh @Louise15926 can you take a look at this, when working on the "task" pkg PR can we remove the dependency from the external errors and use "import errors" instead?

Comment thread internal/pkg/cli/storage_init.go Outdated
Use: "init",
Short: "Creates a new storage table in an environment.",
Use: "init",
Short: "Creates a new storage resource in an environment.",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Short: "Creates a new storage resource in an environment.",
Short: "Creates a new storage config file in the addons directory.",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you think of this? I just want folks to know that it's not going to create a resource.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ooh, yes, good point. Way more clear.

@bvtujo bvtujo added the do-not-merge Pull requests that mergify shouldn't merge until the requester allows it. label Jul 15, 2020
@bvtujo bvtujo requested a review from kohidave July 15, 2020 23:21
@bvtujo bvtujo removed the do-not-merge Pull requests that mergify shouldn't merge until the requester allows it. label Jul 15, 2020
@mergify mergify bot merged commit 65728e6 into aws:master Jul 15, 2020
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.

5 participants