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

setup upstream workflow #13

Merged
merged 8 commits into from
Feb 4, 2024
Merged

setup upstream workflow #13

merged 8 commits into from
Feb 4, 2024

Conversation

bloep
Copy link
Member

@bloep bloep commented Jan 21, 2024

runs pr changes against dependent repositories to receive early feedback.

@bloep bloep requested a review from gharlan January 21, 2024 13:27
coverage: none

- name: set redaxo/php-cs-fixer-config to current branch
run: composer require redaxo/php-cs-fixer-config:dev-${{ github.head_ref }} --no-scripts --no-update --dev
Copy link
Member

Choose a reason for hiding this comment

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

Funktioniert das auch, wenn Externe PRs hier im Repo erstellen, die also in einem Fork liegen?
Passiert hier im Repo bisher nicht wirklich. Daher vielleicht irrelevant. Andererseits wäre es aber gerade da auch interessant, die Auswirkungen zu sehen.

Copy link
Member Author

Choose a reason for hiding this comment

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

Ne, fork würde aktuell nicht gehen. Aber glaube nicht, dass das aktuell notwendig ist

@gharlan
Copy link
Member

gharlan commented Feb 4, 2024

Danke!

@gharlan gharlan merged commit 9b819f5 into main Feb 4, 2024
2 checks passed
@gharlan gharlan deleted the upstream branch February 4, 2024 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants