Skip to content

Conversation

@timholy
Copy link
Member

@timholy timholy commented Aug 2, 2023

For example, without this, iteration over .code yields AbstracChars rather than Chars.

@codecov
Copy link

codecov bot commented Aug 2, 2023

Codecov Report

Merging #337 (0a7b570) into main (ad9b166) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #337   +/-   ##
=======================================
  Coverage   96.56%   96.56%           
=======================================
  Files          14       14           
  Lines        4158     4158           
=======================================
  Hits         4015     4015           
  Misses        143      143           
Files Changed Coverage Δ
src/source_files.jl 97.39% <100.00%> (ø)

@c42f c42f merged commit 84ccafe into JuliaLang:main Aug 3, 2023
@c42f
Copy link
Member

c42f commented Aug 3, 2023

Thanks for the fix, having this abstract was not intentional!

At some point I want to redo SourceFile completely maybe along the lines of #190.

c42f pushed a commit that referenced this pull request Aug 12, 2023
c42f pushed a commit to JuliaLang/julia that referenced this pull request Oct 17, 2025
topolarity pushed a commit to JuliaLang/julia that referenced this pull request Nov 14, 2025
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.

3 participants