Skip to content
This repository has been archived by the owner on Jul 1, 2022. It is now read-only.

fix: Remove drupal/core in favor of core-recommended only #2580

Merged
merged 13 commits into from
Nov 8, 2021

Conversation

podarok
Copy link
Contributor

@podarok podarok commented Nov 8, 2021

Original Issue, this PR is going to fix: https://openy.atlassian.net/browse/MAINTAIN-204

Make sure these boxes are checked before asking for review of your pull request - thank you!

If there is a new feature or this is a bug fix - use 9.x-2.x branch. We'll tag for release if the bug is critical asap or tag for release next bug fix release until critical issue arrived.

Steps for review

General checks

Thank you for your contribution!

@podarok podarok added PR: DO NOT MERGE Work in Progress or something not to be merged at all Type: Bug Bug. Something is not working as intended. labels Nov 8, 2021
@podarok podarok self-assigned this Nov 8, 2021
@gundevel
Copy link
Collaborator

gundevel commented Nov 8, 2021

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3562/

Andrii hates composer since now
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
Still hate composer
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@ymcatwincities ymcatwincities deleted a comment from gundevel Nov 8, 2021
@gundevel
Copy link
Collaborator

gundevel commented Nov 8, 2021

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3565/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3565/

@gundevel
Copy link
Collaborator

gundevel commented Nov 8, 2021

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3565/

@gundevel
Copy link
Collaborator

gundevel commented Nov 8, 2021

Build comment file:

Check Open Y Installation Wizard at http://install.openy.cibox.tools/build3566/install.php


Test PASSed.
Refer to this link for build results (access rights to CI server needed):
http://openy.cibox.tools:8080/job/PR_BUILDER_COMPOSER/3566/

@gundevel
Copy link
Collaborator

gundevel commented Nov 8, 2021

Open Y Upgrade Path site check installed at http://upgrade.openy.cibox.tools/build3566/

@podarok podarok removed the PR: DO NOT MERGE Work in Progress or something not to be merged at all label Nov 8, 2021
@hamrant hamrant added the PR: Code Reviewed Code reviewed. Ready for QA label Nov 8, 2021
@podarok podarok merged commit 981c2da into 9.x-2.x Nov 8, 2021
@podarok podarok deleted the 9.x-2.x-MAINTAIN-204 branch November 8, 2021 14:48
@podarok podarok added Level: Advanced Complex task. Usually for Core team Severity: Critical Critical issue in terms of priorities. Not a security issue. labels Nov 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Level: Advanced Complex task. Usually for Core team PR: Code Reviewed Code reviewed. Ready for QA Severity: Critical Critical issue in terms of priorities. Not a security issue. Type: Bug Bug. Something is not working as intended.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants