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

Auto format on save #815

Merged
merged 5 commits into from
Jan 14, 2016
Merged

Auto format on save #815

merged 5 commits into from
Jan 14, 2016

Conversation

lordmat0
Copy link
Contributor

#810

I wasn't sure if this should be a default on, so I decided to have it defaulted to off.

@basarat
Copy link
Member

basarat commented Jan 13, 2016

so I decided to have it defaulted to off.

Definitely. Don't want user to open someone else's project's typescript file for quick editing just to have it change on them 🌹

@basarat
Copy link
Member

basarat commented Jan 13, 2016

Please answer: Can you try on a project that doesn't have the atom key already. Does it work (the tsconfig.json shouldn't be modifed and format on save shouldn't happen and there should be no errors in the atom debug console)? 🌹

@lordmat0
Copy link
Contributor Author

I tried it already, it works correctly :), been testing on atom-typescript-examples

We create the key here:
https://github.com/lordmat0/atom-typescript/blob/auto_format_on_save/lib/main/tsconfig/tsconfig.ts#L402-L406

Before I use it here:
https://github.com/lordmat0/atom-typescript/blob/auto_format_on_save/lib/main/tsconfig/tsconfig.ts#L445

Would it also not be worth adding it as a menu item? so others using atom on the same project don't have to share the same setting.

@basarat
Copy link
Member

basarat commented Jan 14, 2016

Would it also not be worth adding it as a menu item? so others using atom on the same project don't have to share the same setting.

You mean atom config item? No. Its good to have this shared as a requirement on using the project 🌹

basarat added a commit that referenced this pull request Jan 14, 2016
@basarat basarat merged commit e86a49e into TypeStrong:master Jan 14, 2016
@basarat basarat mentioned this pull request Jan 14, 2016
@basarat
Copy link
Member

basarat commented Jan 14, 2016

@lordmat0 lordmat0 deleted the auto_format_on_save branch January 14, 2016 17:33
@lordmat0 lordmat0 restored the auto_format_on_save branch January 14, 2016 17:33
@s00500
Copy link

s00500 commented Nov 1, 2017

Has this feature been removed again? Why?

@TypeStrong TypeStrong locked and limited conversation to collaborators Jan 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants