Drupal depends on eslint-plugin-yml 0.14.0. However, the latest version is 1.2.0: https://www.npmjs.com/package/eslint-plugin-yml. Release notes: https://github.com/ota-meshi/eslint-plugin-yml/blob/master/CHANGELOG.md

Release notes snippet

eslint-plugin-yml, the YAML style checker for ESLint, has been upgraded from 0.14.0 to 1.2.0.

CommentFileSizeAuthor
#8 3308826-4-REROLL-REROLLED-95x.patch3.15 KBspokje
#4 3308826-4-REROLL-10x.patch3.07 KBbnjmnm
#4 3308826-4-REROLL-95x.patch3.15 KBbnjmnm
#2 3308826-2.patch3.07 KBlauriii

Comments

lauriii created an issue. See original summary.

lauriii’s picture

lauriii
he/him
Primary language Finnish
Location Finland
commented
Status: Active » Needs review
StatusFileSize
new 3308826-2.patch3.07 KB 
$ yarn lint:yaml
yarn run v1.22.17
warning ../../package.json: No license field
$ node ./node_modules/eslint/bin/eslint.js --ext .yml .
✨  Done in 18.74s.
nod_’s picture

nod_
Primary language French
Location Lille
commented
Status: Needs review » Reviewed & tested by the community

works for me

bnjmnm’s picture

bnjmnm
he/him
Primary language English
Location Ann Arbor, MI
commented
StatusFileSize
new 3308826-4-REROLL-95x.patch3.15 KB
new 3308826-4-REROLL-10x.patch3.07 KB

  • bnjmnm committed cfc9682 on 10.1.x 
    Issue #3308826 by bnjmnm, lauriii: Update eslint-plugin-yml to 1.2.0

  • bnjmnm committed 09892af on 10.0.x 
    Issue #3308826 by bnjmnm, lauriii: Update eslint-plugin-yml to 1.2.0
bnjmnm’s picture

bnjmnm
he/him
Primary language English
Location Ann Arbor, MI
commented
Version: 10.0.x-dev » 9.5.x-dev
Status: Reviewed & tested by the community » Patch (to be ported)

Committed to but 10.1.x and cherry picked to 10.0.x
The 9.5 patch uploaded an hour ago already won't apply. Its an eventful day.

spokje’s picture

spokje
Primary language Dutch
commented
StatusFileSize
new 3308826-4-REROLL-REROLLED-95x.patch3.15 KB
spokje’s picture

spokje
Primary language Dutch
commented
Status: Patch (to be ported) » Needs review

Rock-n-Re-roll!

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs review » Reviewed & tested by the community

Reroll is ready to rock.

  • bnjmnm committed 5c6e762 on 9.5.x 
    Issue #3308826 by lauriii, Spokje, nod_, longwave: Update eslint-plugin-...
bnjmnm’s picture

bnjmnm
he/him
Primary language English
Location Ann Arbor, MI
commented
Status: Reviewed & tested by the community » Fixed

Great to hit refresh and see everything we need!
Ported to 9.5.x.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Fixed » Needs work
Issue tags: +Needs release note

This front-end dependency update should probably have a release note.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Issue summary: View changes
Status: Needs work » Needs review
Issue tags: -Needs release note

Not sure what else there is to say, this one barely seems worth mentioning.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Issue tags: +9.5.0 release notes, +10.0.0 release notes
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Needs review » Fixed

That's all I needed. Thanks!

Status: Fixed » Closed (fixed)

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