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

chore: make composer self-update during build #682

Closed
wants to merge 1 commit into from

Conversation

lazysoundsystem
Copy link
Contributor

Refs: OPS-9635

Prompted by https://jenkins.aws.ahconu.org/view/Updates/job/rwint-site-dev-updates/949/console - but also to make it consistent with the other repos.

Copy link

Coverage Report

Totals Coverage
Statements: 11.71% ( 1089 / 9297 )
Methods: 11.57% ( 82 / 709 )
Lines: 11.73% ( 1007 / 8588 )

Copy link

Build output

Composer Validate success

PHP Lint success

Docker Build success

Environment Setup success

Site Install success

PHP Code Sniffer success

Software Versions PHP 8.2.12 (cli) (built: Oct 26 2023 17:33:49) (NTS) Copyright (c) The PHP Group Zend Engine v4.2.12, Copyright (c) Zend Technologies with Zend OPcache v8.2.12, Copyright (c), by Zend Technologies with Xdebug v3.2.1, Copyright (c) 2002-2023, by Derick Rethans Composer version 2.6.5 2023-10-06 10:11:52
Drupal Logs

Pusher: @lazysoundsystem, Action: pull_request, Workflow: Run tests

Copy link
Collaborator

@orakili orakili left a comment

Choose a reason for hiding this comment

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

See comment on ticket for discussion. Marking this PR as blocked for now.

@orakili orakili added the blocked This PR is blocked, do not merge it label Nov 21, 2023
@orakili
Copy link
Collaborator

orakili commented Nov 30, 2023

Closing as per discussion in ticket.

@orakili orakili closed this Nov 30, 2023
@orakili orakili deleted the OPS-9635-composer-self-update branch February 27, 2024 23:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked This PR is blocked, do not merge it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants