mfm.js icon indicating copy to clipboard operation
mfm.js copied to clipboard

Use MfmUrl AST in MfmLink AST

Open shibaobun opened this issue 2 years ago • 2 comments

What

  • Use MfmUrl AST in MfmLink AST
  • Add test cases for reproducting bracket urls within links

Why

  • This fixes issues with brackets disappearing
  • Closes #83

Additional info (optional)

This is my first time using typescript, so please let me know if there's anything that can be changed for best practice! Thank you!

shibaobun avatar Oct 15 '22 17:10 shibaobun

Codecov Report

Merging #126 (88daccc) into develop (ae92c2a) will increase coverage by 0.00%. The diff coverage is 100.00%.

:exclamation: Current head 88daccc differs from pull request most recent head e7f70c6. Consider uploading reports for the commit e7f70c6 to get more accurate results

@@           Coverage Diff            @@
##           develop     #126   +/-   ##
========================================
  Coverage    97.72%   97.73%           
========================================
  Files            8        8           
  Lines         1541     1544    +3     
  Branches       317      233   -84     
========================================
+ Hits          1506     1509    +3     
- Misses          33       34    +1     
+ Partials         2        1    -1     
Impacted Files Coverage Δ
src/internal/parser.ts 99.33% <100.00%> (ø)
src/internal/util.ts 97.63% <100.00%> (ø)
src/node.ts 100.00% <100.00%> (ø)
src/api.ts 100.00% <0.00%> (ø)
src/internal/index.ts 100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

codecov[bot] avatar Oct 18 '22 15:10 codecov[bot]

@syuilo i've rebased this PR with your changes, please let me know if there's anything you'd like changed!

shibaobun avatar Feb 05 '23 16:02 shibaobun