Skip to content

Conversation

@c42f
Copy link
Member

@c42f c42f commented Nov 7, 2023

Ensure that we never treat things like "mutable\nstruct" as introducing a mutable struct definition, even within parentheses where newline whitespace is insignificant.

Fixes part of #380

Ensure that we never treat things like `"mutable\nstruct"` as a mutable
struct definition, even within parentheses where newline whitespace is
insignificant.
@c42f c42f mentioned this pull request Nov 7, 2023
10 tasks
@codecov
Copy link

codecov bot commented Nov 7, 2023

Codecov Report

Merging #386 (db0f2ee) into main (1351d70) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #386   +/-   ##
=======================================
  Coverage   96.58%   96.58%           
=======================================
  Files          14       14           
  Lines        4183     4183           
=======================================
  Hits         4040     4040           
  Misses        143      143           
Files Coverage Δ
src/parser.jl 98.10% <100.00%> (ø)

@c42f c42f merged commit 0df34c9 into main Nov 7, 2023
@c42f c42f deleted the caf/fix-contextual-keyword-pairs-in-parens branch November 7, 2023 11:29
c42f added a commit that referenced this pull request Nov 12, 2023
Ensure that we never treat things like `"mutable\nstruct"` as a mutable
struct definition, even within parentheses where newline whitespace is
insignificant.
c42f added a commit to JuliaLang/julia that referenced this pull request Oct 17, 2025
…iaLang/JuliaSyntax.jl#386)

Ensure that we never treat things like `"mutable\nstruct"` as a mutable
struct definition, even within parentheses where newline whitespace is
insignificant.
topolarity pushed a commit to JuliaLang/julia that referenced this pull request Nov 14, 2025
…iaLang/JuliaSyntax.jl#386)

Ensure that we never treat things like `"mutable\nstruct"` as a mutable
struct definition, even within parentheses where newline whitespace is
insignificant.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant