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

Add jsfmt package to javascript lang layer. #3525

Closed
wants to merge 1 commit into from

Conversation

geksilla
Copy link
Contributor

Added plugin jsfmt.el to javascript layer and installation instruction.

@geksilla
Copy link
Contributor Author

@TheBB, @StreakyCobra, can you check this one when you'll have an opportunity? Thanks!

@robbyoconnor
Copy link
Contributor

👍

1 similar comment
@ghost
Copy link

ghost commented Feb 18, 2016

👍

@syl20bnr syl20bnr self-assigned this Feb 18, 2016
@laduke
Copy link

laduke commented Oct 23, 2016

Is there something I could do to help this get merged? Was it just the README conflict that stopped it?
Using jsfmt could possible reduce the amount of incoming "JSX / ES6 / ES7 indenting doesn't work" issues.

Provide option to activate jsfmt by default.
Added keybindng to toggle jsfmt mode.
Added installation and configuration example to Readme.
@geksilla
Copy link
Contributor Author

geksilla commented Nov 4, 2016

Resolved merge conflicts.

@Compro-Prasad
Copy link
Contributor

@syl20bnr Any updates on this?

@robbyoconnor
Copy link
Contributor

robbyoconnor commented Jun 23, 2018 via email

@Compro-Prasad
Copy link
Contributor

Compro-Prasad commented Sep 12, 2018

I think jsfmt is already there in react layer but there is no mention of the package in packages.el. We also have web-beautify which should be enough to format things. @sdwolfz This can be closed if we already have jsfmt else we can ask @geksilla to update this PR.

Copy link

@bjartur bjartur left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there any specific reason to call jsfmt rather than calling esformat directly? Is it a matter of compatibility with jsfmt configuration files in existing projects?

@robbyoconnor
Copy link
Contributor

@geksilla Rebase needed :)

@github-actions
Copy link

github-actions bot commented Oct 3, 2020

This PR has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Please let us know if this PR is still need merging!

@github-actions github-actions bot added the stale marked as a stale issue/pr (usually by a bot) label Oct 3, 2020
@github-actions github-actions bot closed this Jan 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Package stale marked as a stale issue/pr (usually by a bot)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants