This is a sub-issue of #1938580: [META] Make active config save format match the default yml file (order and quotes). Check meta issue for Steps to reproduce and Coding standards

Files need to be fixed
breakpoint.settings.yml

Remaining tasks

CommentFileSizeAuthor
#2 breakpoint-active-config-1942110-2.patch389 bytessidharthap
#1 breakpoint-active-config-1942110-1.patch0 bytessidharthap
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

vijaycs85’s picture

vijaycs85
Location London, UK
CreditAttribution: vijaycs85 commented
Issue summary: View changes

Updating header

sidharthap’s picture

sidharthap
Primary language English
CreditAttribution: sidharthap commented
FileSize
breakpoint-active-config-1942110-1.patch0 bytes

Here is the patch file.

sidharthap’s picture

sidharthap
Primary language English
CreditAttribution: sidharthap commented
FileSize
breakpoint-active-config-1942110-2.patch389 bytes

previous file shows 0kb. here is the new one.

vijaycs85’s picture

vijaycs85
Location London, UK
CreditAttribution: vijaycs85 commented
Status: Active » Needs review
vijaycs85’s picture

vijaycs85
Location London, UK
CreditAttribution: vijaycs85 commented

Looks matching with active file...

Status: Needs review » Needs work
Issue tags: -Configuration system, -Config novice

The last submitted patch, breakpoint-active-config-1942110-2.patch, failed testing.

vijaycs85’s picture

vijaycs85
Location London, UK
CreditAttribution: vijaycs85 commented
Status: Needs work » Needs review

#2: breakpoint-active-config-1942110-2.patch queued for re-testing.

vijaycs85’s picture

vijaycs85
Location London, UK
CreditAttribution: vijaycs85 commented
Issue tags: +Configuration system, +Config novice

#2: breakpoint-active-config-1942110-2.patch queued for re-testing.

vijaycs85’s picture

vijaycs85
Location London, UK
CreditAttribution: vijaycs85 commented
Status: Needs review » Reviewed & tested by the community

Looks good to go.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed dd91e31 and pushed to 8.x. Thanks!

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

(not verified) CreditAttribution: commented
Issue summary: View changes

Updated issue summary.