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.

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

finatra: Patch round-13 code #2210

Closed
wants to merge 1 commit into from
Closed

finatra: Patch round-13 code #2210

wants to merge 1 commit into from

Conversation

cacoco
Copy link
Contributor

@cacoco cacoco commented Aug 20, 2016

Hi there! There are some improvements we'd like to get into the round-13 testing run for Finatra based on the released preliminary results.

It's not clear if this patch should be done against the current round-13 branch or the round-14 branch. This PR is based on the current round-14 branch. Please let me know if I need to re-do against a different branch to get this into the round-13 testing.

Problem

There are some improvements we'd like to get into the round-13
testing run.

Solution

Add the improvements.
@knewmanTE
Copy link
Contributor

@cacoco sorry for all of the branching confusion! We're currently using master to represent our preview/final testing for round 13 and round-14 to represent all of the larger tests and features that will be included in our round 14 testing.

Would you mind closing this merge request and re-open it against master?

@cacoco
Copy link
Contributor Author

cacoco commented Aug 23, 2016

@knewmanTE sure thing. Thanks for the info. Can you tell me when the window will close for patching round 13? Thanks again!

@cacoco cacoco closed this Aug 23, 2016
@cacoco cacoco mentioned this pull request Aug 27, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants