search

sinhrks / ggfortify

Define fortify and autoplot functions to allow ggplot2 to handle some popular R packages.
Other
527 stars 65 forks source link

Removed gglagplot in favor of forecast::gglagplot #171

Closed terrytangyuan closed 6 years ago

terrytangyuan commented 7 years ago

@sinhrks what do you think? Any other preferred solution to fix Travis test and for the package going forward?

coveralls commented 7 years ago

Coverage Status

Coverage decreased (-0.07%) to 93.626% when pulling fbdffc2694cccad46d05054752a7c2740cc69baa on gglagplot into bd301bf6b36e580eb98e89e8ff8a750689f6ccbb on master.

terrytangyuan commented 7 years ago

@sinhrks Is it okay to remove this? This currently conflicts with the one in forecast, which is why Travis is failing.

terrytangyuan commented 7 years ago

Ping! @sinhrks Lots of builds failed on CRAN. https://cran.r-project.org/web/checks/check_results_ggfortify.html

Could you change the setting on PR merging so I can move this forward?

terrytangyuan commented 7 years ago

@sinhrks Could you take a look?