Skip to content

Conversation

@Ayesh
Copy link
Member

@Ayesh Ayesh commented Mar 18, 2025

This drops the FranzDiebold/github-env-vars-action in favor of native GH-Actions replacement patterns available.

Notable replacements:

  • ${{ env.CI_REPOSITORY_OWNER_SLUG }}: $(echo '${{ github.repository }}' | tr '[:upper:]' '[:lower:]')
  • ${{ env.CI_REPOSITORY_NAME }}: ${{ github.event.repository.name }}

Related: #9639 and linkorb/php-docker-base#16.

  • Breaking change: fix or feature that would cause existing functionality to not work as expected

Checklist

@Ayesh Ayesh requested a review from mhitza as a code owner March 18, 2025 19:05
Copy link
Collaborator

@mhitza mhitza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. I'd like the variable usage to be extended.

I've removed your breaking change flag for this change. I'd reserve that for when all changes stabilize and a v1 version becomes necessary.

This drops the `FranzDiebold/github-env-vars-action` in favor of native
GH-Actions replacement patterns available.

Notable replacements:

 - `${{ env.CI_REPOSITORY_OWNER_SLUG }}`: `$(echo '${{ github.repository }}' | tr '[:upper:]' '[:lower:]')`
 - `${{ env.CI_REPOSITORY_NAME }}`: `${{ github.event.repository.name }}`

Related: [#9639](http://team.linkorb.com/cards/9639) and linkorb/php-docker-base#16.
@Ayesh
Copy link
Member Author

Ayesh commented Mar 19, 2025

@mhitza thanks, I covered two more hardcoded linkorb/ prefixes with github.repository.

I think that was all of them.

@Ayesh Ayesh requested a review from mhitza March 19, 2025 02:16
@mhitza mhitza merged commit f817de1 into linkorb:main Mar 19, 2025
3 checks passed
@Ayesh Ayesh deleted the ci/9641-drop-github-env-vars-action branch March 19, 2025 13:24
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