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.

Sign up for GitHub

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

Jump to bottom

doc: add reporting bugs doc and small nit improvements in the README #1480

Merged
merged 2 commits into from
Oct 6, 2020
Merged

doc: add reporting bugs doc and small nit improvements in the README #1480

merged 2 commits into from
Oct 6, 2020

Conversation

camilamacedo86
Copy link
Contributor

doc: add reporting bugs doc and small nit improvements in the README

@camilamacedo86 camilamacedo86 mentioned this pull request Apr 24, 2020
Merged
@camilamacedo86
Copy link
Contributor Author

c/c @njhale

@camilamacedo86
Copy link
Contributor Author

/test e2e-gcp

@camilamacedo86
Copy link
Contributor Author

/test e2e-gcp
/test e2e-aws-olm

@camilamacedo86
Copy link
Contributor Author

/test e2e-gcp

@camilamacedo86
Copy link
Contributor Author

@njhale wdyt about this one? is it ok to be merged? Could you please give a hand with?

@camilamacedo86
Copy link
Contributor Author

/test e2e-gcp

@camilamacedo86
Copy link
Contributor Author

/test e2e-gcp-upgrade
/test e2e-gcp

@camilamacedo86
Copy link
Contributor Author

/test e2e-aws-olm

@camilamacedo86
Copy link
Contributor Author

/test e2e-gcp-upgrade

@camilamacedo86
Copy link
Contributor Author

/test e2e-gcp

@Bowenislandsong
Copy link
Member

Thanks for doing this.
/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 8, 2020
@Bowenislandsong
Copy link
Member

/retest

@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Jun 9, 2020
@camilamacedo86
Copy link
Contributor Author

Hi @Bowenislandsong,

I rebased it with the master again to see if the CI issue is fixed.

@Bowenislandsong
Copy link
Member

/retest

@Bowenislandsong
Copy link
Member

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Jun 9, 2020
njhale
njhale reviewed Jul 16, 2020
View reviewed changes
README.md Outdated Show resolved Hide resolved
@camilamacedo86
Copy link
Contributor Author

Hi @njhale,

Could we move with this one now?

camilamacedo86
camilamacedo86 commented Aug 12, 2020
View reviewed changes
README.md Outdated Show resolved Hide resolved
camilamacedo86
camilamacedo86 commented Aug 12, 2020
View reviewed changes
README.md Outdated Show resolved Hide resolved
@openshift-ci-robot openshift-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Aug 12, 2020
@njhale
Copy link
Member

njhale commented Aug 12, 2020

/approve
/lgtm
/retest

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Aug 12, 2020
@ecordell
Copy link
Member

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Aug 13, 2020
@camilamacedo86
Copy link
Contributor Author

/test e2e-aws-olm

@camilamacedo86
Copy link
Contributor Author

/test e2e-gcp

@camilamacedo86
Copy link
Contributor Author

/test e2e-aws-olm

@camilamacedo86
Copy link
Contributor Author

camilamacedo86 commented Aug 14, 2020 •
edited
Loading

/test e2e-gcp-upgrade
/test e2e-aws-olm
/test e2e-gcp

@ecordell @njhale @Bowenislandsong I already rebased with the master and re-started this test many times. It is just a change in the README. In POV should not call the jobs indeed since is just a doc change. however, I do not know why it does not pass in the CI. Has no reason or relation with the changes done here :-(

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@camilamacedo86
Copy link
Contributor Author

/test e2e-gcp-upgrade
/test e2e-gcp

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

5 similar comments
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@camilamacedo86
Copy link
Contributor Author

/test e2e-gcp

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

1 similar comment
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@njhale njhale added the kind/documentation Categorizes issue or PR as related to documentation. label Sep 17, 2020
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

3 similar comments
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@camilamacedo86
Copy link
Contributor Author

/test e2e-upgrade
/test e2e-aws-console-olm
/test e2e-gcp
/test e2e-aws-olm

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

3 similar comments
@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-bot
Copy link
Contributor

/retest

Please review the full test history for this PR and help us cut down flakes.

@openshift-merge-robot openshift-merge-robot merged commit f795a8c into operator-framework:master Oct 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Bowenislandsong Bowenislandsong Awaiting requested review from Bowenislandsong

@njhale njhale Awaiting requested review from njhale

Assignees

@ecordell ecordell

@njhale njhale

@Bowenislandsong Bowenislandsong

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. kind/documentation Categorizes issue or PR as related to documentation. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@camilamacedo86 @Bowenislandsong @njhale @openshift-ci-robot @of-deploy-bot @openshift-bot @ecordell @openshift-merge-robot