Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: added CreateRole command completion #655

Closed
wants to merge 0 commits into from

Conversation

joeyhage
Copy link
Contributor

@joeyhage joeyhage commented May 4, 2022

No description provided.

@joeyhage
Copy link
Contributor Author

joeyhage commented Oct 3, 2022

@mpilquist saw that you are a maintainer now - this should be a very straightforward PR. Thanks in advance for your time!

@mpilquist
Copy link
Member

Looks good, thanks! Is there any way to add some test coverage?

@joeyhage
Copy link
Contributor Author

joeyhage commented Oct 3, 2022

@mpilquist I was rebasing and reset my fork to the main branch so GH auto closed the PR. New PR with test coverage is here: #714. Thank you!

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.

2 participants