RFR: 8282944: GHA: Add Alpine Linux x86_64 pre-integration check [v11]
George Adams
gdams at openjdk.org
Mon Aug 19 15:27:51 UTC 2024
On Mon, 19 Aug 2024 13:21:42 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
>> George Adams has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - improve conditional logic
>> - try this
>
> .github/workflows/main.yml line 84:
>
>> 82:
>> 83: # List of platforms to exclude by default
>> 84: EXCLUDED_PLATFORMS=("alpine-linux-x64")
>
> I'm not to fond of declaring a list like this in the middle of a code body. Perhaps you can set it up as a non-required workflow input with alpine as the default value, declared next to the default list of platforms?
@magicus do you want me to push this change?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20577#discussion_r1721971949
More information about the build-dev
mailing list