Skip to content

Request to add matlab-actions/setup-matlab Action to allow list #643

Merged
raboof merged 1 commit intoapache:mainfrom
sgilmore10:add-setup-matlab-action
Apr 1, 2026
Merged

Request to add matlab-actions/setup-matlab Action to allow list #643
raboof merged 1 commit intoapache:mainfrom
sgilmore10:add-setup-matlab-action

Conversation

@sgilmore10
Copy link
Copy Markdown
Member

@sgilmore10 sgilmore10 commented Mar 31, 2026

Overview

Action name: setup-matlab
Action URL: https://github.com/matlab-actions/setup-matlab
Version to pin: aa8bbc7b76daa63c5d456d1430cbd6cb5b626ab4

Action Description

From the Setup MATLAB README.md:

The Setup MATLAB action enables you to set up MATLAB® and other MathWorks® products on a GitHub®-hosted (Linux®, Windows®, or macOS) runner or self-hosted UNIX® (Linux or macOS) runner.

Motivation

The apache/arrow repository uses matlab-actions/setup-matlab and matlab-actions/run-tests, which I will add to the allow list in a separate PR, to build and test the MATLAB Apache Arrow bindings. However, beginning on March 20th, the MATLAB CI checks have not been able to run because matlab-actions/setup-matlab is not on the allow list.

Related Actions

I will create a separate PR to add matlab-actions/run-tests to the allow list.


Checklist

  • The action is listed in the GitHub Actions Marketplace
  • The action is not already on the list of approved actions
  • The action has a sufficient number of contributors or has contributors within the ASF community (It is actively maintained by MathWorks/has 9 contributors)
  • The action has a clearly defined license
  • The action is actively developed or maintained
  • The action has CI/unit tests configured
  • Compiled JavaScript in dist/ matches a clean rebuild (verify with uv run utils/verify-action-build.py org/repo@hash) (In progress)

@sgilmore10 sgilmore10 marked this pull request as ready for review April 1, 2026 01:20
@raboof
Copy link
Copy Markdown
Member

raboof commented Apr 1, 2026

verified .js files, previously trusted, seems sensible, is running with only read permissions

@raboof raboof merged commit 7f2d7ab into apache:main Apr 1, 2026
7 checks passed
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.

2 participants