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

Switch formatting to reformatter.el #51

Merged
merged 2 commits into from
Jan 1, 2023

Conversation

joachimschmidt557
Copy link
Member

@joachimschmidt557 joachimschmidt557 commented Jan 2, 2021

See #39 for the corresponding issue.

This changes some of the weird behavior which arises from using revert-buffer after formatting such as #49.

Of course, this massively changes formatting behavior for users used to the formatting functionality. For example, formatting on save is now handled by a new minor mode instead of a defcustom and a toggle function. This also adds a dependency on the reformatter package in MELPA.

Closes #39
Closes #61
Closes #49

@joachimschmidt557
Copy link
Member Author

See #75: I will merge the reformatter branch on the 1st of January 2023.

@joachimschmidt557 joachimschmidt557 changed the title Proposal: Switch formatting to reformatter.el Switch formatting to reformatter.el Dec 16, 2022
@joachimschmidt557 joachimschmidt557 merged commit 4356724 into ziglang:master Jan 1, 2023
@joachimschmidt557 joachimschmidt557 deleted the reformatter branch January 1, 2023 08:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant