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

Add header stating its not usable module and remove other sections #4

Closed
wants to merge 8 commits into from

Conversation

3h4x
Copy link

@3h4x 3h4x commented Aug 6, 2020

what

  • Add header stating its not usable module and remove other sections

why

  • Its confusing to see repo like terraform-opsgenie with only example module until first PR is merged

@3h4x 3h4x requested a review from osterman August 6, 2020 13:42
@3h4x 3h4x requested a review from a team as a code owner August 6, 2020 13:42
@3h4x 3h4x requested a review from Gowiem August 6, 2020 13:42
Gowiem
Gowiem previously approved these changes Aug 6, 2020
Copy link
Member

@osterman osterman left a comment

Choose a reason for hiding this comment

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

Thanks for opening. Will review in a bit - but want to change wording.

README.yaml Outdated Show resolved Hide resolved
Co-authored-by: Nuru <Nuru@users.noreply.github.com>
@mergify
Copy link

mergify bot commented Feb 4, 2021

This pull request is now in conflict. Could you fix it @3h4x? 🙏

@nitrocode nitrocode requested review from a team as code owners June 7, 2021 22:50
@mergify mergify bot dismissed osterman’s stale review June 7, 2021 22:50

This Pull Request has been updated, so we're dismissing all reviews.

@nitrocode
Copy link
Member

/test all

nitrocode
nitrocode previously approved these changes Jun 7, 2021
@mergify
Copy link

mergify bot commented Aug 21, 2021

This pull request is now in conflict. Could you fix it @3h4x? 🙏

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.

6 participants