Skip to content

Add support for multi-letter uppercase sigils#12448

Merged
whatyouhide merged 6 commits into
mainfrom
al/multiletter-sigils
Mar 5, 2023
Merged

Add support for multi-letter uppercase sigils#12448
whatyouhide merged 6 commits into
mainfrom
al/multiletter-sigils

Conversation

@whatyouhide

Copy link
Copy Markdown
Member

First pass at #12447.

Still need to update docs 🙃

Comment thread lib/elixir/lib/code/formatter.ex Outdated
whatyouhide and others added 2 commits March 5, 2023 12:49
Co-authored-by: José Valim <jose.valim@dashbit.co>
@whatyouhide

Copy link
Copy Markdown
Member Author

@josevalim ok, this is ready for review. I won't change the docs too much if that's ok, since I'd like to do a small overhaul of the sigil docs in general, since now they're a bit buried inside the Macro docs.

@whatyouhide whatyouhide marked this pull request as ready for review March 5, 2023 18:53
@whatyouhide

Copy link
Copy Markdown
Member Author

@josevalim ok, this is ready for review. I won't change the docs too much if that's ok, since I'd like to do a small overhaul of the sigil docs in general, since now they're a bit buried inside the Macro docs.

@whatyouhide whatyouhide requested a review from josevalim March 5, 2023 18:53
@whatyouhide whatyouhide merged commit e74a798 into main Mar 5, 2023
@whatyouhide whatyouhide deleted the al/multiletter-sigils branch March 5, 2023 19:10
leandrocp added a commit to leandrocp/monaco-editor that referenced this pull request Jun 24, 2023
Elixir v1.15.0 has added support for multi-letter
uppercase sigils, eg: ~LVN|content|

Refs:
PR - elixir-lang/elixir#12448
Release notes - https://github.com/elixir-lang/elixir/releases/tag/v1.15.0
leandrocp added a commit to leandrocp/monaco-editor that referenced this pull request Jun 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants