Skip to content
This repository was archived by the owner on Apr 5, 2018. It is now read-only.

Jira 143#25

Open
GAZDOWN wants to merge 2 commits intoleapp-to:masterfrom
GAZDOWN:jira-143
Open

Jira 143#25
GAZDOWN wants to merge 2 commits intoleapp-to:masterfrom
GAZDOWN:jira-143

Conversation

@GAZDOWN
Copy link
Contributor

@GAZDOWN GAZDOWN commented Oct 20, 2017

leapp-to/snactor#55 should be merged before this one.

}

c := executor.New("remote-target-check-group", actorInput)
c := executor.New("remote-target-check-group-v2", actorInput)
Copy link
Contributor

Choose a reason for hiding this comment

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

Why do you want to add -v2? Can't you change actual actor? Is it because something different is using remote-target-check-group "v1"?

}

c := executor.New("port-mapping", string(actorInput))
c := executor.New("port-mapping-v2", string(actorInput))
Copy link
Contributor

Choose a reason for hiding this comment

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

the same question as above

@mfranczy
Copy link
Contributor

mfranczy commented Oct 23, 2017

Ah I'm reading the description from JIRA.. so right now ignore my comments. I have to understand what you wrote there and then I will remove my questions or I will continue asking :)

@pirat89
Copy link

pirat89 commented Dec 5, 2017

Guys, we should really talk about practices of PR. It's nice to mention, that there is some Jira for internals, but the PR should carry complete info about its purpose so it will be available for everyone in future, who look at here.

@vinzenz
Copy link
Contributor

vinzenz commented Dec 5, 2017

@pirat89 Yeah actually I said so - just we did not write it here - Basically the JIRA tickets shoudn't actually be referenced here at all due to the fact that they aren't openly accessible

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants