Add scheduled test to Actions for all features

[jlucktay]

Pull Request

Related issue

Fixes #3506.

What does this PR do?

Add a new job to the Rust workflow to run cargo build and cargo test (on the cron schedule only) with the --all-features flag.
This will execute across all three environments: Linux, macOS, Windows.

Autoformat the Rust workflow file via the Red Hat YAML extension for Visual Studio Code.
This straightens out whitespace and string quoting for safer parsing.

As pointed out by @irevoire here, changes to CI such as this one will need to wait for #3496 before going ahead.
The new action was executed on my fork but ended up failing on some metrics tests, as called out in that linked comment.

PR checklist

Please check if your PR fulfills the following requirements:

• Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
• Have you read the contributing guidelines?
• Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

[curquiza]

Hello @jlucktay

Thanks a lot for the PR! Small changes needed

• can you only test this on Ubuntu (not windows and macOS) like we do in test-linux job? We use an Ubuntu container and not the native ubuntu-18.04 of GitHub Action that will be soon deprecated
• can you remove the empty line between each step of the job you added?

[curquiza]

Hello @jlucktay

Are you sure your CI is working, have you done any manual test?

Because I don't see the following steps in your jobs:

- name: Install needed dependencies
        run: |
          apt-get update && apt-get install -y curl
          apt-get install build-essential -y
- uses: actions-rs/toolchain@v1
        with:
          toolchain: stable
          override: true

And they are in the other jobs using the ubuntu-18.04 Docker image

[jlucktay]

Hello @jlucktay

Are you sure your CI is working, have you done any manual test?

Because I don't see the following steps in your jobs:

- name: Install needed dependencies
        run: |
          apt-get update && apt-get install -y curl
          apt-get install build-essential -y
- uses: actions-rs/toolchain@v1
        with:
          toolchain: stable
          override: true

And they are in the other jobs using the ubuntu-18.04 Docker image

I triggered some runs on my fork about two weeks ago before I opened the PR, yes.

The Actions runner images come with Rust, curl, and the build-essential package installed already, so those extra two steps do seem redundant, but I have included them now anyway.

I have just run another fresh round of tests now.

[curquiza]

I triggered some runs on my fork about two weeks ago before I opened the PR, yes.

Yes so you tested 2 weeks ago with ubuntu-18.04 provided by GitHub and not the changes I asked with the docker image ubuntu-18.04. Because it does not work without these 2 steps 😊

Thanks for the changes, I review this during the afternoon

[curquiza]

Thanks @jlucktay

Looks good to me!

bors merge

[bors]

🕐 Waiting for PR status (GitHub check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set.

[curquiza]

bors merge

[bors]

Stopped waiting for PR status (GitHub check) without running due to duplicate requests to run. You may check Bors to see that this PR is included in a batch by one of the other requests.

[github-actions]

Uffizzi Preview deployment-18245 was deleted.

[bors]

Build succeeded:

• Run Clippy
• Run Rustfmt
• Run tests in debug
• Tests on macos-12
• Tests on ubuntu-18.04
• Tests on windows-2022