Skip to content
This repository was archived by the owner on Jul 30, 2021. It is now read-only.

Conversation

@robszumski
Copy link
Collaborator

@robszumski robszumski commented Feb 5, 2019

  • Update readme
  • Move docs from readme into separate files
  • Stub out testing doc linked from readme
  • Broaden annotation doc to all required fields
  • Use a standardized category list
    • community
    • RH products
    • certified ISVs

@openshift-ci-robot openshift-ci-robot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Feb 5, 2019
@robszumski
Copy link
Collaborator Author

Fixed, PTAL again

@robszumski
Copy link
Collaborator Author

Updated the categories everywhere. If we need to remove any docs, happy to do that in a follow up PR.

@SamiSousa SamiSousa mentioned this pull request Feb 8, 2019
@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 11, 2019
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 12, 2019
@robszumski
Copy link
Collaborator Author

Rebased, would love to get this merged before I have a bunch of conflicts again

Copy link
Contributor

@SamiSousa SamiSousa 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-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 12, 2019
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Feb 12, 2019
@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Feb 12, 2019
Copy link
Contributor

@SamiSousa SamiSousa left a comment

Choose a reason for hiding this comment

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

/lgtm

@SamiSousa SamiSousa merged commit 0773ee2 into operator-framework:master Feb 12, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants