Skip to content

Adding fragment file for PR #3650 (Rescue tasks reconciliation)#3729

Merged
jmrodri merged 3 commits intooperator-framework:masterfrom
VenkatRamaraju:fragments
Aug 13, 2020
Merged

Adding fragment file for PR #3650 (Rescue tasks reconciliation)#3729
jmrodri merged 3 commits intooperator-framework:masterfrom
VenkatRamaraju:fragments

Conversation

@VenkatRamaraju
Copy link
Copy Markdown
Contributor

Description of the change:
Added the fragment file for #3650 (Missed in this PR).


kind: "addition"

breaking: false
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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


kind: "bugfix"

breaking: false No newline at end of file
Copy link
Copy Markdown
Contributor

@camilamacedo86 camilamacedo86 Aug 12, 2020

Choose a reason for hiding this comment

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

we need to give a space ^ nit . See the ^

Copy link
Copy Markdown
Contributor

@camilamacedo86 camilamacedo86 left a comment

Choose a reason for hiding this comment

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

/lgtm cancel

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

kind: "bugfix"

breaking: false No newline at end of file
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Since it was not added in the same PR we need to fill : pull_request_override: with the PR number.

@camilamacedo86
Copy link
Copy Markdown
Contributor

Just add the PR number in: pull_request_override: 0 and leave a space at the end of the file.
Otherwise all fine.

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

@fabianvf fabianvf 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 Aug 13, 2020
Copy link
Copy Markdown
Contributor

@camilamacedo86 camilamacedo86 left a comment

Choose a reason for hiding this comment

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

/lgtm

@jmrodri jmrodri merged commit 28a8caa into operator-framework:master Aug 13, 2020
joelanford pushed a commit to joelanford/operator-sdk that referenced this pull request Sep 17, 2020
…onciliation) (operator-framework#3729)

* Added fragments file

* Fragment file

* Changes
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.

5 participants