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

ACMS-3354: Remove 10.2.x DIT patches from Acquia CMS Common module. #1683

Merged
merged 9 commits into from
Dec 6, 2023

Conversation

rajeshreeputra
Copy link
Contributor

Motivation
Fixes #3354

Proposed changes
Remove Drupal core patches as available in 10.2.x core.

Alternatives considered
NA

Testing steps

  • Checkout branch ACMS-3354
  • Install dependencies using composer install
  • verify patches applied successfully.

Merge requirements

  • Major change, Minor change, Bug, Enhancement, and/or Chore label applied
  • Manual testing by a reviewer

@chandan-singh7929
Copy link
Collaborator

@rajeshreeputra - Overall changes looks good to me, but I am getting error on my local when running ./vendor/bin/acms si
Screenshot 2023-11-23 at 6 01 25 PM

@chandan-singh7929
Copy link
Collaborator

Also, lets make sure that our CI run on next minor as well to see the actual failure for this PR.
cc @vishalkhode1

@rajeshreeputra rajeshreeputra force-pushed the ACMS-3354 branch 2 times, most recently from 44feb88 to 3a8b307 Compare November 28, 2023 05:58
@rajeshreeputra rajeshreeputra force-pushed the ACMS-3354 branch 2 times, most recently from 2d0621c to 8805947 Compare November 28, 2023 07:40
@chandan-singh7929 chandan-singh7929 force-pushed the ACMS-3354 branch 3 times, most recently from 9347e29 to 397edfc Compare November 28, 2023 09:23
@rajeshreeputra rajeshreeputra force-pushed the ACMS-3354 branch 4 times, most recently from 8390fd8 to d48fbd6 Compare November 28, 2023 11:48
@rajeshreeputra rajeshreeputra changed the base branch from develop to feature/drupal10.2.x November 29, 2023 12:16
ACMS-3354: Ristrict ACMS Common 3.3.x to drupal core 10.1.x version.
@vishalkhode1 vishalkhode1 force-pushed the ACMS-3354 branch 4 times, most recently from 01fbedd to 6381603 Compare November 29, 2023 15:45
@rajeshreeputra rajeshreeputra force-pushed the ACMS-3354 branch 2 times, most recently from 10cb310 to de9efd5 Compare December 1, 2023 05:49
@vishalkhode1 vishalkhode1 merged commit 56f3b0b into feature/drupal10.2.x Dec 6, 2023
23 checks passed
@vishalkhode1 vishalkhode1 deleted the ACMS-3354 branch December 6, 2023 11:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants