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 Erlang alignment regexps #1728

Merged
merged 2 commits into from Aug 16, 2018
Merged

Add Erlang alignment regexps #1728

merged 2 commits into from Aug 16, 2018

Conversation

davidw
Copy link
Contributor

@davidw davidw commented Feb 26, 2018

These allow for the alignment of maps and records with the Emacs align command.

I think this may be a better, more 'emacsy' way of doing things than this:

#1707

These allow for the alignment of maps and records
@dgud
Copy link
Contributor

dgud commented Mar 2, 2018

Is there some initialization missing or is this new functionality?
I get "File mode specification error: (void-variable align-rules-list)" in emacs-24.5 and we want to
work with emacs-24.1.

I assume this replaces #1707

@dgud dgud mentioned this pull request Mar 5, 2018
@dgud
Copy link
Contributor

dgud commented Mar 5, 2018

When testing on some files I noticed that it's missing ':=' for maps.

@fenollp
Copy link
Contributor

fenollp commented Mar 5, 2018

Hi there, noticed this patch brings some regressions now that #1681 is merged:
https://travis-ci.org/fenollp/erlang-formatter/builds/349321370
Maybe your regexps do not match that recent patch?

@davidw
Copy link
Contributor Author

davidw commented May 7, 2018

@fenollp I'm looking, but... I don't see how my patch and that one even interact? My patch is for maps and records. I'm not familiar with that test suite - is there some advice on how I could run it locally to check?

@dgud dgud changed the base branch from master to maint August 13, 2018 13:29
@dgud dgud added the testing currently being tested, tag is used by OTP internal CI label Aug 14, 2018
@dgud dgud merged commit 0d5c753 into erlang:maint Aug 16, 2018
@dgud
Copy link
Contributor

dgud commented Aug 16, 2018

Merged, Thanks

velimir added a commit to velimir/otp that referenced this pull request Aug 28, 2018
erlang-mode crashes with the following error:
Symbol’s value as variable is void: align-rules-list

caused by erlang#1728
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants