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

Remove more deprecated features #508

Merged
merged 20 commits into from
Nov 12, 2017
Merged

Remove more deprecated features #508

merged 20 commits into from
Nov 12, 2017

Conversation

mvz
Copy link
Contributor

@mvz mvz commented Nov 9, 2017

Summary

Remove remaining deprecated bits in preparation of the 1.0.0 release.

Details

This removes

  • Code marked @deprecated
  • Methods and hooks that warn about being deprecated
  • Scenarios marked (deprecated) and the functionality they test

I have kept one deprecated scenario; the one for setting the home directory to 'some other path'. It is related to #478 and it is not clear to me what Aruba 1.0.0 will and will not support here.

Motivation and Context

A major version bump is the right time to remove deprecated items.

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Update documentation

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.

@mvz
Copy link
Contributor Author

mvz commented Nov 11, 2017

Ready for review.

Copy link
Contributor

@olleolleolle olleolleolle left a comment

Choose a reason for hiding this comment

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

This change is quite easy to review, as each of the removed section was clearly labeled deprecated. And, you've described what deprecated feature is not removed.

LGTM!

@mvz mvz merged commit 2fa7ded into master Nov 12, 2017
@mvz mvz deleted the remove-more-deprecated-features branch November 12, 2017 09:52
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.

2 participants