Skip to content

Conversation

@ewgRa
Copy link
Contributor

@ewgRa ewgRa commented Nov 10, 2015

Q A
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #12643
License MIT
Doc PR

@ewgRa
Copy link
Contributor Author

ewgRa commented Nov 10, 2015

When fabbot.io patch is applied, there is a conflicts, I think better to merge it without conflicts.

@ewgRa
Copy link
Contributor Author

ewgRa commented Nov 10, 2015

Test failure seems not related to this PR

Copy link
Member

Choose a reason for hiding this comment

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

no need to ref this one in the code

@nicolas-grekas
Copy link
Member

@ewgRa could you please fabbot's issues also (even if unrelated to your change)?

@nicolas-grekas
Copy link
Member

See #12643 (comment) for some background on why this works

@nicolas-grekas
Copy link
Member

👍 (after rebasing)

@ewgRa
Copy link
Contributor Author

ewgRa commented Nov 10, 2015

done, anything else?

@nicolas-grekas
Copy link
Member

Thank you @ewgRa.

nicolas-grekas added a commit that referenced this pull request Nov 10, 2015
This PR was squashed before being merged into the 2.3 branch (closes #16510).

Discussion
----------

[2.3][Process] fix Proccess run with pts enabled

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #12643
| License       | MIT
| Doc PR        |

Commits
-------

9cf90fb [2.3][Process] fix Proccess run with pts enabled
@nicolas-grekas
Copy link
Member

Closed via ab3c8f8

@ewgRa ewgRa deleted the issue-12643 branch November 10, 2015 10:52
This was referenced Nov 23, 2015
This was referenced Nov 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants