Skip to content

Conversation

@aram535
Copy link
Contributor

@aram535 aram535 commented Dec 14, 2017

Changed:

  • Put back API getAllProjects()
  • fixed JUnit for getAllProjects()
  • On a vanilla system since the order of tests cannot be guaranteed some of the count tests fail. Changed the Junit to simply check for exception rather than actual project numbers.
  • Changed the next version to 1.2.9-SNAPSHOT (1.2.8 is on maven)

@slonopotamus
Copy link

@aram535 commit 9b08228 didn't remove getAllProjects, you wanted to point to something different.

@timols timols merged commit cb51a38 into timols:master Jan 2, 2018
@aram535
Copy link
Contributor Author

aram535 commented Jan 5, 2018

@slonopotamus you're right it wasn't the commit but it was the merge that removed the method. So that's still on the author of the patch to fix prior to committing the MR. No matter the cause the fix is it.

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.

3 participants