Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 492919

Summary: Support reading .eslintrc.yml and .eslintrc.yaml files
Product: [ECD] Orion Reporter: Michael Rennie <Michael_Rennie>
Component: JS ToolsAssignee: Olivier Thomann <Olivier_Thomann>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: curtis.windatt.public, Michael_Rennie, Olivier_Thomann, Silenio_Quarti
Version: 12.0   
Target Milestone: 13.0   
Hardware: PC   
OS: Mac OS X   
Whiteboard:
Attachments:
Description Flags
Proposed patch
none
Proposed patch none

Description Michael Rennie CLA 2016-05-03 14:07:34 EDT
A follow up to bug 461228 - we lack the proper libraries / support to consume a yml or yaml file. To be able to support these in the linting setting we would likely need something like js-yaml (https://github.com/nodeca/js-yaml) or the like.
Comment 1 Michael Rennie CLA 2016-06-08 10:58:39 EDT
*** Bug 495698 has been marked as a duplicate of this bug. ***
Comment 2 Olivier Thomann CLA 2016-07-15 11:54:56 EDT
The corresponding CQ is https://dev.eclipse.org/ipzilla/show_bug.cgi?id=11684
Comment 3 Olivier Thomann CLA 2016-07-15 12:57:03 EDT
Patch is ready for this. Waiting for the CQ. The example given in bug 495698 can now be properly parsed, but eslint on that project still doesn't work as it requires bug 496027 to be fixed.
Comment 4 Olivier Thomann CLA 2016-07-15 13:33:06 EDT
Created attachment 263135 [details]
Proposed patch
Comment 5 Olivier Thomann CLA 2016-07-15 13:34:10 EDT
As usual, the patch cannot be applied back to the master stream. Working on it.
Comment 6 Olivier Thomann CLA 2016-07-15 13:39:03 EDT
Created attachment 263137 [details]
Proposed patch

This patch applies successfully.
Comment 8 Olivier Thomann CLA 2016-08-15 11:37:32 EDT
In order to get Silenio's case to work, we need to adopt eslint 3.0.1. With that commit, .yaml and .yml files are properly parsed.