deprecate aliasses

[richierocks]

To fix #180

Work in progress; do not merge.

[ismayc]

Will be shifting to get_p_value() instead. Could some suggestions be added in case the aliases are attempted to be used going forward? Essentially something like

Error: `p_value()` is now deprecated. Please use `get_p_value()` instead.

[richierocks]

@ismayc I've made all the required fixes to use get_confidence_intervals() and get_p_value().

It uses base::.Deprecated() to give a standard warning if you try get_pvalue() or one of the other deprecated functions.

Warning message:
'get_pvalue' is deprecated.
Use 'get_p_value' instead.
See help("Deprecated")

There is a warning on the travis-ci build that I can't get to the bottom of. It claims that get_pvalue() is an undefined export.

Everything works fine on my local machine, so I'd be tempted to just merged then rebuild the documentation.

[richierocks]

Hmm. I realized that this PR still thinks it is taking content from the master branch. I'll close this a and give you a new Pr using the work from the develop branch.

[github-actions]

This pull request has been automatically locked. If you believe you have found a related problem, please file a new issue (with a reprex: https://reprex.tidyverse.org) and link to this issue.