action-shellcheck icon indicating copy to clipboard operation
action-shellcheck copied to clipboard

feat: support ARM runners

Open LesnyRumcajs opened this issue 9 months ago • 9 comments

Sample run: https://github.com/LesnyRumcajs/action-shellcheck/actions/runs/13140996529

Adds support for ARM-based runners.

Closes https://github.com/ludeeus/action-shellcheck/issues/112

LesnyRumcajs avatar Feb 04 '25 17:02 LesnyRumcajs

Even though arm runners are super common, you're probably better off forking the repo and adding support for ARM there.

bwhitehead0 avatar Feb 04 '25 19:02 bwhitehead0

Even though arm runners are super common, you're probably better off forking the repo and adding support for ARM there.

Could you please elaborate why this would be a better solution? Different architecture is already supported by the upstream tool. I'd find it confusing if I had to use different actions for different runners.

LesnyRumcajs avatar Feb 04 '25 20:02 LesnyRumcajs

The maintainer has zero interest in adding ARM support last I and others checked, unfortunately.

bwhitehead0 avatar Feb 04 '25 21:02 bwhitehead0

@ludeeus Would you like make things easier for others and add support for ARM runners? Seems it'd be appreciated by the general community, especially since ARM runners will likely become the standard, given their benefits over AMD ones.

LesnyRumcajs avatar Feb 04 '25 21:02 LesnyRumcajs

The maintainer has zero interest in adding ARM support last I and others checked, unfortunately.

Where did i say that?

ludeeus avatar Feb 05 '25 06:02 ludeeus

@ludeeus Would you like make things easier for others and add support for ARM runners? Seems it'd be appreciated by the general community, especially since ARM runners will likely become the standard, given their benefits over AMD ones.

Once github hosted runners are out of preview, this can be added.

ludeeus avatar Feb 05 '25 06:02 ludeeus

Once github hosted runners are out of preview, this can be added.

That seems reasonable; thanks for the quick reply. I'll ping you once this happens. Cheers.

LesnyRumcajs avatar Feb 05 '25 08:02 LesnyRumcajs

The maintainer has zero interest in adding ARM support last I and others checked, unfortunately.

Where did i say that?

In the PR I submitted a year or two back.

Private/custom runners have supported ARM for a long time. Enterprises use them a lot.

bwhitehead0 avatar Feb 05 '25 17:02 bwhitehead0

For completeness that PR points to this comment:

Untill GitHub provide action runners for it, this is not sometthing I will add.

ludeeus avatar Feb 05 '25 17:02 ludeeus