Skip to content

Update linux-ci for macos and all ci for action updates for Node.js#159

Open
jhmgoossens wants to merge 2 commits intocoin-or:masterfrom
jhmgoossens:dev-ci-202604-master
Open

Update linux-ci for macos and all ci for action updates for Node.js#159
jhmgoossens wants to merge 2 commits intocoin-or:masterfrom
jhmgoossens:dev-ci-202604-master

Conversation

@jhmgoossens
Copy link
Copy Markdown
Contributor

  • Update linux-ci to add the new macos-26 and macos-26-intel runners.
    -- Simplify the combinations of macos and compilers (gcc / clang)
    -- Reduce the number of MaxOS builds to only the two most recent MacOS, and intel only for the latest MacOS
    -- Remove "OSX=xx" flags. It seems not used?
    -- Remove "arch: arm64" for macos-14. It seems not used?
    See also https://docs.github.com/en/actions/reference/runners/github-hosted-runners

  • Update update all ci.yml for several actions to recent versions that use Node 24 rather than the old Node 20. See recent Action warnings.

See also COIN-OR-OptimizationSuite issue 36 discussion.

@jhmgoossens
Copy link
Copy Markdown
Contributor Author

The windows-ci.yml for masters and stables of many coin-or projects use the

      - name: Set up msvc
        if: ${{ matrix.arch == 'msvc' }}
        uses: ilammy/msvc-dev-cmd@v1

But this action msvc-dev-cmd still uses Node.js 20. This results in Warnings in the action Runs like:

Node.js 20 actions are deprecated. The following actions are running on Node.js 20 
and may not work as expected: ilammy/msvc-dev-cmd@v1.

I propose that instead of uses: ilammy/msvc-dev-cmd@v1 we use uses: egor-tensin/vs-shell@v2. This vs-shell action is light-weight and does not use Node.js. Test runs show that such runs work and produce the same binaries. The maintainer is active, even if this action itself has not had recent releases.

See also COIN-OR-OptimizationSuite Issue 37

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant