Test new integration tests. No new release.

[ablaom]

This follows up #1079 which did not actual trigger integration tests because the source of the PR was not dev.

This PR still needs:

• Add new model descriptors to fix doc-generation fail #1084

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (4b6303d) 57.89% compared to head (848d415) 57.89%.
Report is 21 commits behind head on master.

❗ Current head 848d415 differs from pull request most recent head 7358186. Consider uploading reports for the commit 7358186 to get more accurate results

Additional details and impacted files

@@           Coverage Diff           @@
##           master    #1080   +/-   ##
=======================================
  Coverage   57.89%   57.89%           
=======================================
  Files           2        2           
  Lines          38       38           
=======================================
  Hits           22       22           
  Misses         16       16           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ablaom]

Okay, my attempt here in ci.yml to change the MLJ_TEST_INTEGRATION flag for a PR with dev as source does not seem to have worked (julia 1 test is on 6mins, we are expected ~60min).

@DilumAluthge Are you able to look over my ci.yml logic? This is not a strong point of mine.

[DilumAluthge]

Let me take a closer look. Something seems to not be working.

[DilumAluthge]

#1081

[ablaom]

Well, this is weird:

[DilumAluthge]

There are two jobs: push and pull_request. The screenshot you posted is from a push job, in which the various PR-related variables won't be defined. Those variables will be defined in the pull_request job.

[ablaom]

Go it! Yay, it's working!!

Thanks @DilumAluthge ❤️

[ablaom]

Hooray 🎉 . MLJ integration tests are now, well... integrated.