laminas-mvc-skeleton
laminas-mvc-skeleton copied to clipboard
Prepare for Renovate with reusable workflows
| Q | A |
|---|---|
| QA | yes |
Description
Adding Renovate to Laminas, Mezzio repositories. https://gist.github.com/weierophinney/49ca8ae2b916d898dd3d4162b26ff41b
- Configure minimum supported PHP version
- Update composer dependencies
- Update to reusable
Continuous Integrationworkflow - Update to reusable
Automatic Releasesworkflow
Skeleton app is an exception which needs to drop lock file entirely and switch to a scheduled testing.
@Xerkus should I skip this PR?
@ghostwriter No. Dependencies range in composer.json still need to be bumped to a version supported by the lower php bound
It was more of a reminder to @Ocramius comment that skeleton can not have lock file.
There is nothing to run here. phpunit is not installed by default and existing test is rather a placeholder that also wants laminas/laminas-test as dependency.
laminas-test could be installed by installer prompt but real test needs to test the installer process itself to be of any use for the CI.
There is nothing to run here. phpunit is not installed by default and existing test is rather a placeholder that also wants laminas/laminas-test as dependency.
I agree, as you previously mentioned, this is a skeleton repo, feel free to close this.
laminas-test could be installed by installer prompt but real test needs to test the installer process itself to be of any use for the CI.
yes, could be addressed in a separate pr.
by the way,xmllint failing, it may be a relative path issue. (unrelated to this pr)
I don't think this can use the reusable workflow since all the tooling configuration is for use in a new project and not meant for checking this repository.
In fact, I think you could remove the file at .github/workflows/continuous-integration.yml and then merge this with a different title.
I've updated the PR and Title to reflect our discussion.
@Ocramius do you still want to target a new major with this?
Skeleton isn't really affected by BC policy anyway :shrug:
Thanks @ghostwriter