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

Revert "Delete classes that are deprecated for six years." #988

Merged
merged 1 commit into from
Sep 12, 2014

Conversation

marcphilipp
Copy link
Member

This reverts commit 745ca05. The
deletion of the classes causes some problems for users (see
https://groups.yahoo.com/neo/groups/junit/conversations/topics/24572).

This revert is independent from #749. #749 was fixed by 88f7a12.


Since @stefanbirkner seems to be on vacation (are you?) this pull requests continues based on his work in #977.


/**
* @deprecated Included for backwards compatibility with JUnit 4.4. Will be
* removed in the next release. Please use
Copy link
Member

Choose a reason for hiding this comment

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

next major release?

ditto in other files

@kcooney
Copy link
Member

kcooney commented Sep 12, 2014

Thanks! One suggestion.

Might be nice to squash some of those commits, but not a big deal

@marcphilipp
Copy link
Member Author

Fixed the missing "majors". Now squashing...

This reverts commit 745ca05. The
deletion of the classes caused some problems for users (see
https://groups.yahoo.com/neo/groups/junit/conversations/topics/24572).

In addition, the new coding style has been applied and "major" has
been added to the deprecation messages in the Javadoc.
@marcphilipp
Copy link
Member Author

Re-pushed the branch with squashed commits.

@kcooney Please take another look.

kcooney added a commit that referenced this pull request Sep 12, 2014
Revert "Delete classes that are deprecated for six years."
@kcooney kcooney merged commit 7d07b8d into junit-team:master Sep 12, 2014
@kcooney
Copy link
Member

kcooney commented Sep 12, 2014

awesome! thanks!

@marcphilipp marcphilipp deleted the revert-gh862 branch December 7, 2014 18:49
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