Page MenuHomePhabricator

Have libup rename .eslintrc to .eslintrc.json
Closed, ResolvedPublic0 Estimated Story Points

Description

.eslintrc (if they exist) files should automatically be renamed to .eslintrc.json.

See https://gerrit.wikimedia.org/r/c/labs/libraryupgrader/+/523340 for an example patch that implemented a different kind of fixer. You likely want to use the shutil.move function to implement this.

Affects: https://codesearch.wmflabs.org/search/?q=%5BA-z%5D&i=nope&files=.eslintrc%24&repos=

Event Timeline

Legoktm triaged this task as Lowest priority.Jul 15 2019, 10:50 PM
Legoktm created this task.

Hi @Legoktm ,
it's .eslintrc files under libraryupgrader, right?

Thanks

Hi @Legoktm ,
it's .eslintrc files under libraryupgrader, right?

Thanks

hi! libraryupgrader clones *other* git repositories and applies some fixes to them. We want libraryupgrader to automatically rename .eslintrc files if they exist. The related patch I linked might help as well.

Thank you @Legoktm .
I will work on this.

@Mh-3110 any progress so far? Let me know if you need any help/assistance or have questions.

@Legoktm ,
It is clear what needs to be done here. Will submit the patch this week-end the latest.

Will let you know if I need any assistance.

Thank you

@Mh-3110 I'm unassigning this from you since you haven't made any progress - feel free to reassign if you're ready to work on it again.

Change 563467 had a related patch set uploaded (by Umherirrender; owner: Umherirrender):
[labs/libraryupgrader@master] Use json file extension for the config files

https://gerrit.wikimedia.org/r/563467

Change 563467 merged by jenkins-bot:
[labs/libraryupgrader@master] Use json file extension for the config files

https://gerrit.wikimedia.org/r/563467

Legoktm assigned this task to Umherirrender.