Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jan 20, 2022. It is now read-only.

Conversation

@MansurAshraf
Copy link
Contributor

Created a separate sub-project 'summingbird-core-test'

Copy link
Collaborator

Choose a reason for hiding this comment

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

This is missing new line at end of file, package statement at the top and the apache header

@johnynek
Copy link
Collaborator

man it would be cool if our tests worked.

@sritchie
Copy link
Collaborator

@johnynek Come to me with those concerns - Q2 is already fully booked, but I'll take it into consideration for the Q3 roadmap.

@johnynek
Copy link
Collaborator

@sritchie too real, man.

@MansurAshraf MansurAshraf mentioned this pull request Feb 17, 2015
@johnynek
Copy link
Collaborator

@MansurAshraf can we close this one now?

@MansurAshraf
Copy link
Contributor Author

@johnynek no I need this merged for internal refactoring we are doing.

@MansurAshraf
Copy link
Contributor Author

To give a little context I need TestGraphs published as an artifact as multiple downstream projects are dependent on it.

@johnynek
Copy link
Collaborator

Okay, yes. Got it. Let me review.

ianoc added a commit that referenced this pull request Mar 17, 2015
refactored core-tests into separate sub-project
@ianoc ianoc merged commit 8450259 into twitter:develop Mar 17, 2015
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.

5 participants