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

Fix README.md #225

Merged
merged 1 commit into from
May 23, 2023
Merged

Fix README.md #225

merged 1 commit into from
May 23, 2023

Conversation

tmshort
Copy link
Contributor

@tmshort tmshort commented May 19, 2023

Fixes #219

Make it consistent with the Makefile

Description

Reviewer Checklist

  • API Go Documentation
  • Tests: Unit Tests (and E2E Tests, if appropriate)
  • Comprehensive Commit Messages
  • Links to related GitHub Issue(s)

README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@tmshort tmshort force-pushed the readme2 branch 3 times, most recently from 024ce43 to a3ed066 Compare May 22, 2023 15:12
@tmshort
Copy link
Contributor Author

tmshort commented May 22, 2023

ping @everettraven

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Fixes operator-framework#219

Make it consistent with the Makefile

Signed-off-by: Todd Short <tshort@redhat.com>
Copy link
Contributor

@everettraven everettraven left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label May 23, 2023
@tmshort tmshort merged commit c859404 into operator-framework:main May 23, 2023
5 checks passed
@tmshort tmshort added this to the v0.2.0 milestone May 23, 2023
@tmshort tmshort deleted the readme2 branch August 25, 2023 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Documentation discrepancy over make run and make install
2 participants