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

feat: rework the nodesToString function to output expected element tags #234

Merged
merged 7 commits into from
Jul 8, 2022

Conversation

cheton
Copy link
Member

@cheton cheton commented Jul 7, 2022

@cheton cheton changed the title feat: rework the nodes-to-string function to output correct tags feat: rework the nodes-to-string function to output correct tags Jul 7, 2022
@codecov
Copy link

codecov bot commented Jul 7, 2022

Codecov Report

Merging #234 (5b8a7e5) into master (1d8294d) will increase coverage by 0.27%.
The diff coverage is 94.87%.

@@            Coverage Diff             @@
##           master     #234      +/-   ##
==========================================
+ Coverage   90.65%   90.92%   +0.27%     
==========================================
  Files           6        6              
  Lines         535      562      +27     
==========================================
+ Hits          485      511      +26     
- Misses         50       51       +1     
Impacted Files Coverage Δ
src/index.js 98.14% <75.00%> (+0.07%) ⬆️
src/nodes-to-string.js 85.24% <96.77%> (+5.24%) ⬆️
src/parser.js 90.19% <100.00%> (+0.09%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1d8294d...5b8a7e5. Read the comment docs.

@cheton cheton self-assigned this Jul 7, 2022
@cheton cheton changed the title feat: rework the nodes-to-string function to output correct tags feat: rework the nodesToString function to output expected element tags Jul 7, 2022
@cheton cheton merged commit 9e91b0c into master Jul 8, 2022
@cheton cheton deleted the i18next-scanner-229 branch July 8, 2022 12:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant