search

lalvim / PartialLeastSquaresRegressor.jl

Implementation of a Partial Least Squares Regressor
MIT License
40 stars 8 forks source link

Test maintenance #30

Closed ablaom closed 1 year ago

ablaom commented 1 year ago

It's been a long time since CI ran on this repo. In the meantime, several test dependencies have had breaking changes and CI is now failing (for me locally - travis appears to be dead). In this PR we:

For reviewing, please note that the last commit contains a lot of whitespace fixes, so best to incrementally review, rather than look at "files changed".

ablaom commented 1 year ago

@filipebraida @lalvim This was quite a bit of work. Be good if you can take a look.

ablaom commented 1 year ago

Okay, I also see that CI is no longer active, because it's travis. I suggest we add a GH action for this. I can work on a PR to add this and TagBot, but please ping me to let know you're okay with this.

ablaom commented 1 year ago

@lalvim Can we please have a new release tagged? It's been more than 7 months since I made this contribution. I'm still hitting the same errors in my MLJ integration tests and this is causing some maintenance headaches.