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

Adds action to automatically update composer dependencies #17969

Merged
merged 1 commit into from Oct 17, 2021

Conversation

sgiehl
Copy link
Member

@sgiehl sgiehl commented Sep 6, 2021

Description:

As our composer requirements are almost all configured to only allow minor or patch release updates, it should be fine to run this action weekly in order to check if any updates are available.

The first PR created after merging this might be a bit bigger, as we didn't update the dependencies for a while.

Updates to major version will still need to be done manually, as they require a change in composer.json

Review

@sgiehl sgiehl added Needs Review PRs that need a code review Better processes Indicates an issue is about improving how we work. labels Sep 6, 2021
@Findus23
Copy link
Member

Findus23 commented Sep 6, 2021

Maybe this could also run a composer outdated so that one can immediately see from the log if there is anything that needs major upgrades?

@sgiehl
Copy link
Member Author

sgiehl commented Sep 6, 2021

Maybe this could also run a composer outdated so that one can immediately see from the log if there is anything that needs major upgrades?

We need to find another process for major updates. Maybe another action could automatically create issues for packages that have a major update. See matomo-org/developer-documentation#547

@tsteur tsteur modified the milestones: 4.6.0, 4.7.0 Sep 6, 2021
@github-actions
Copy link
Contributor

This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers

@github-actions github-actions bot added the Stale The label used by the Close Stale Issues action label Sep 14, 2021
@sgiehl sgiehl added Do not close PRs with this label won't be marked as stale by the Close Stale Issues action and removed Stale The label used by the Close Stale Issues action labels Sep 14, 2021
@github-actions
Copy link
Contributor

This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers

@github-actions github-actions bot added Stale The label used by the Close Stale Issues action and removed Stale The label used by the Close Stale Issues action labels Sep 22, 2021
@github-actions
Copy link
Contributor

This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers

@github-actions github-actions bot added Stale The label used by the Close Stale Issues action and removed Stale The label used by the Close Stale Issues action labels Sep 30, 2021
@github-actions
Copy link
Contributor

github-actions bot commented Oct 8, 2021

This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers

@github-actions github-actions bot added Stale The label used by the Close Stale Issues action and removed Stale The label used by the Close Stale Issues action labels Oct 8, 2021
@tsteur
Copy link
Member

tsteur commented Oct 14, 2021

@sgiehl feel free to merge when you're back

@sgiehl sgiehl merged commit 6bb1569 into 4.x-dev Oct 17, 2021
@sgiehl sgiehl deleted the composerupdateaction branch October 17, 2021 20:10
@sgiehl sgiehl modified the milestones: 4.7.0, 4.6.0 Nov 19, 2021
@justinvelluppillai justinvelluppillai added the not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org. label Nov 29, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Better processes Indicates an issue is about improving how we work. Do not close PRs with this label won't be marked as stale by the Close Stale Issues action Needs Review PRs that need a code review not-in-changelog For issues or pull requests that should not be included in our release changelog on matomo.org.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants