Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GitHub Actions shared composite actions #8

Merged
merged 20 commits into from
Sep 1, 2021
Merged

Conversation

zmughal
Copy link
Member

@zmughal zmughal commented Sep 1, 2021

Creates a set of composite actions for use across the organization. This is
based on the workflows in the https://github.com/PDLPorters/pdl repository.

Of note are the workarounds that use shell script to support conditional steps
by using if ${{ toJSON( <boolean expression> ) }}; then ...; fi since the
if: key is not yet supported per actions/runner#834.
This works because the output of toJSON() in that case is either true or
false which are the same as the shell script built-ins.

This does cause some issues as Windows does not work perfectly with shell: bash, but there are workarounds for that as well.

This is a first draft of this approach and there will be a set of associated
PRs applying this to different repositories.

zmughal added 20 commits August 27, 2021 16:20
This uses a composite action with conditionals, but these will not yet
work per <actions/runner#834>.

This commit is just to show the start of the action before transforming
it using a shell script workaround. If simpler, that transformation can
be reverted to this version when support for conditionals in composite
actions is added.
This is for testing before merging the new workflow into the pdl
repository.
This is a workaround for composite actions since they do not support
conditionals yet.
Apparently GitHub Actions uses non-POSIX compliant environment variable
names (!!!) for INPUT_* variables.

See <https://github.community/t/dashes-in-docker-action-inputs-produce-unreadable-env-vars/117382/7>,
<actions/toolkit#219>.
This is to simplify use in a build matrix.
So that it sits inside the conditional.
@zmughal zmughal merged commit d7ee38d into master Sep 1, 2021
@zmughal zmughal deleted the gha-shared-actions branch September 1, 2021 03:12
zmughal added a commit to PDLPorters/pdl that referenced this pull request Sep 1, 2021
zmughal added a commit to PDLPorters/PDL-Stats that referenced this pull request Sep 1, 2021
Uses the shared actions introduced by <PDLPorters/devops#8>.
zmughal added a commit that referenced this pull request Sep 1, 2021
This uses yamllint to check the GitHub Actions action.yml files.

Uses the shared actions introduced by <#8>.
zmughal added a commit that referenced this pull request Sep 1, 2021
This uses yamllint to check the GitHub Actions action.yml files.

Uses the shared actions introduced by <#8>.
zmughal added a commit that referenced this pull request Sep 1, 2021
This uses yamllint to check the GitHub Actions action.yml files.

Uses the shared actions introduced by <#8>.
zmughal added a commit to PDLPorters/PDL-Fit-OO that referenced this pull request Dec 22, 2022
zmughal added a commit to PDLPorters/PDL-Parallel-threads that referenced this pull request Dec 23, 2022
zmughal added a commit to PDLPorters/PDL-Fit-ExpRate that referenced this pull request Dec 23, 2022
zmughal added a commit to PDLPorters/PDL-Graphics-ColorSpace that referenced this pull request Dec 23, 2022
zmughal added a commit to PDLPorters/PDL-Fit-ExpRate that referenced this pull request Dec 23, 2022
zmughal added a commit to PDLPorters/PDL-Parallel-threads that referenced this pull request Dec 23, 2022
zmughal added a commit to PDLPorters/App-Prima-REPL that referenced this pull request Dec 23, 2022
zmughal added a commit to PDLPorters/App-Prima-REPL that referenced this pull request Dec 23, 2022
zmughal added a commit to PDLPorters/PDL-OpenCV that referenced this pull request Dec 23, 2022
zmughal added a commit to PDLPorters/PDL-Graphics-Prima that referenced this pull request Dec 23, 2022
zmughal added a commit to PDLPorters/App-Prima-REPL that referenced this pull request Dec 23, 2022
zmughal added a commit to PDLPorters/PDL-Graphics-Prima that referenced this pull request Dec 24, 2022
zmughal added a commit to PDLPorters/PDL-Graphics-Prima that referenced this pull request Dec 24, 2022
zmughal added a commit to PDLPorters/PDL-OpenCV that referenced this pull request Dec 24, 2022
zmughal added a commit to PDLPorters/PDL-OpenCV that referenced this pull request Dec 24, 2022
zmughal added a commit to PDLPorters/PDL-Parallel-threads that referenced this pull request Dec 25, 2022
zmughal added a commit to PDLPorters/PDL-OpenCV that referenced this pull request Dec 26, 2022
zmughal added a commit to zmughal/pdl-finance-ta that referenced this pull request Dec 28, 2022
zmughal added a commit to zmughal/pdl-finance-ta that referenced this pull request Dec 28, 2022
zmughal added a commit to zmughal/pdl-finance-ta that referenced this pull request Dec 28, 2022
zmughal added a commit to PDLPorters/PDL-Graphics-Prima that referenced this pull request Dec 30, 2022
zmughal added a commit to zmughal-contrib/Alien-Gnuplot that referenced this pull request Feb 17, 2023
zmughal added a commit to zmughal-contrib/Alien-Gnuplot that referenced this pull request Feb 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant