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

Automatic vscode settings migration #133

Closed
nicojs opened this issue Aug 2, 2019 · 5 comments · Fixed by #250
Closed

Automatic vscode settings migration #133

nicojs opened this issue Aug 2, 2019 · 5 comments · Fixed by #250
Labels
status: accepting prs Please, send in a PR to resolve this! ✨ type: enhancement New feature or request

Comments

@nicojs
Copy link

nicojs commented Aug 2, 2019

why you no .vscode settings? 😒

🚀 Feature Request

It would be awesome if this tool could migrate .vscode settings from tslint to eslint. For example:

Change this:

"editor.codeActionsOnSave": {
    "source.fixAll.tslint": true
}

To:

"eslint.autoFixOnSave": true

Vscode settings can reside in either .vscode/settings.json or a *.vscode-workspace file.

Existing Behavior

No vscode support, bro. 🥺

Change Proposal

Probably this should be under a compiler flag.

Also, thanks for this awesome tool 👏, I will check it out later when I have some time.

@JoshuaKGoldberg
Copy link
Member

Strongly agreed with this use case. Here's a rough proposal:

  • Add CLI option to specify an editor config path, and have it default to ./vscode/settings.json
  • Rename the method passed to runCli.ts to convertConfigs (plural)
  • Write that new convertConfigs that does both the existing convertLintConfig work and convertEditorConfig

--editor, perhaps?

I'd like to keep the logic here editor-agnostic to users, so after this is done, we can file a followup issue for Atom support. :atom:

@JoshuaKGoldberg JoshuaKGoldberg added good first issue Good for newcomers; welcome aboard! status: accepting prs Please, send in a PR to resolve this! ✨ type: enhancement New feature or request labels Aug 2, 2019
@nicojs
Copy link
Author

nicojs commented Aug 21, 2019

Wait, there are other editors? 😨

I hope to find some time soon, but I'm usually swamped... Whoever invented the 24h day?

@MrCube42
Copy link
Contributor

MrCube42 commented Oct 5, 2019

@JoshuaKGoldberg I'll give it a try tomorrow. Looks like a nice task to implement while keeping everything clean in the code base.

@MrCube42
Copy link
Contributor

@JoshuaKGoldberg @nicojs I started working on this with WIP-PR (#250). It was quite hard for me to fit into the existing structure and using this kind of DI ;) However finally I got a first version into working. There are still some things left (see TODOs).

Maybe you should have a look at it first if this approach would suite your needs?

@JoshuaKGoldberg
Copy link
Member

It was quite hard for me to fit into the existing structure and using this kind of DI ;)

Ahh, yes, welcome to the light side. It takes a bit to get used to 😝

@JoshuaKGoldberg JoshuaKGoldberg removed the good first issue Good for newcomers; welcome aboard! label Dec 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: accepting prs Please, send in a PR to resolve this! ✨ type: enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants