#19 Complete the deprecation annotations already started
Closed: fixed 5 years ago Opened 5 years ago by emaldonado.

When building jss there are many deprecation warnings. Though many cannot be dealt with, and don't need to, at the present we should complete what has been started already and properly annotate the remaining ones.
Adding the pertinent "@deprecation" and "@Deprecation" annotations with some comments will make the maintainers life easier in the future and is a fairly safe thing to do.

This is a spin-off of https://pagure.io/jss/issue/17 where it's sated that the effort should be done in two phases: 1) Add annotations the comments - no more than that and 2) Address to issues as the need arise in the future when things get removed. Filing this issue to concentrate on phase 1. Will do a pull request for the easy part.


Note to the reviewer: When testing you should build with 'export CHECK_DEPRECATION=1' which will result in many warnings as this is only annotations. Addressing the warnings is deferred to a later time. The warnings should only be ones regarding deprecations and no other ones.

Crude method I used to test, does same steps before and after the patch

# Build Before

cd ~/jss <--- wrong, cit should be 'cd jss'

This is a new version of my testing procedure in the form of a shell strip. It produces a lot of files , specially diff's of before and after the patch. Hopefully it will make it easier to view. One can also use meld. Will also attach some results next.

HowElioTested.sh

Revised version of the patch

Revised version of the patch

Complete deprecation annotations already started

- Addresses https://pagure.io/jss/issue/19
- Removed to depreactions as requestes on the review
- see https://github.com/dogtagpki/jss/pull/26
- https://github.com/dogtagpki/jss/pull/26#pullrequestreview-159596877
- No deprecation annotions added to org/mozilla/jss/tests/Constants.java
- until we clarify some issues

PR26.patch

For review before I attempt doing a a revised pull request

This was fixed by jss PR#26. This was included in master by commit f38f62af.

Metadata Update from @cipherboy:
- Custom field component adjusted to None
- Custom field feature adjusted to None
- Custom field origin adjusted to None
- Custom field proposedmilestone adjusted to None
- Custom field proposedpriority adjusted to None
- Custom field reviewer adjusted to None
- Custom field type adjusted to None
- Custom field version adjusted to None
- Issue close_status updated to: fixed
- Issue status updated to: Closed (was: Open)

5 years ago

Metadata Update from @cipherboy:
- Issue assigned to cipherboy

5 years ago

Login to comment on this ticket.

Metadata
Attachments 7
Attached 5 years ago View Comment
Attached 5 years ago View Comment
Attached 5 years ago View Comment
Attached 5 years ago View Comment
Attached 5 years ago View Comment
Attached 5 years ago View Comment