laminas / laminas-db

Database abstraction layer, SQL abstraction, result set abstraction, and RowDataGateway and TableDataGateway implementations
https://docs.laminas.dev/laminas-db/
BSD 3-Clause "New" or "Revised" License
119 stars 66 forks source link

Prepare for Renovate with reusable workflows #239

Closed ghostwriter closed 2 years ago

ghostwriter commented 2 years ago
Q A
QA yes

Description

Adding Renovate to Laminas, Mezzio repositories. https://gist.github.com/weierophinney/49ca8ae2b916d898dd3d4162b26ff41b

ghostwriter commented 2 years ago

Unable to locate package php8.1-sqlsrv

Ocramius commented 2 years ago

Looks like checks didn't run, perhaps due to YAML structure/parsing error?

ghostwriter commented 2 years ago

@Ocramius apparently, the services keyword is not supported yet for reusable workflows.

https://docs.github.com/en/actions/using-workflows/reusing-workflows#supported-keywords-for-jobs-that-call-a-reusable-workflow

Ocramius commented 2 years ago

Ooof, guess we'd keep our phpunit tasks written down manually here, while excluding them from the laminas-ci integration perhaps?

ghostwriter commented 2 years ago

Removed reusable Continuous Integration workflow for now.

weierophinney commented 2 years ago

I wonder why no actions triggered with my last push...

Ocramius commented 2 years ago

@weierophinney github partial outage

Ocramius commented 2 years ago

We need to do something like pushing + squashing an empty commit here

weierophinney commented 2 years ago

Yes... But waiting for github status to go green (actions still has degraded performance).

On Wed, Mar 23, 2022, 12:20 PM Marco Pivetta @.***> wrote:

We need to do something like pushing + squashing an empty commit here

— Reply to this email directly, view it on GitHub https://github.com/laminas/laminas-db/pull/239#issuecomment-1076599353, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAAGKV7L6HDC726PV2POX43VBNHGTANCNFSM5O5NRITQ . You are receiving this because you were mentioned.Message ID: @.***>

weierophinney commented 2 years ago

@ghostwriter I'll figure out why the CI container isn't working, and get things sorted. Your work is done here; thanks for the patch!

weierophinney commented 2 years ago

@ghostwriter You overwrote a change I'd pushed to .laminas-ci.json:

    "ignore_php_platform_requirements": {
        "8.1": true
    }

That will allow deps to install in 8.1, which should allow tests to pass.

weierophinney commented 2 years ago

Thanks, @ghostwriter !