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

test: include all dompurify test cases #37

Merged
merged 6 commits into from
Dec 14, 2023
Merged

Conversation

JounQin
Copy link
Member

@JounQin JounQin commented Dec 13, 2023

No description provided.

@JounQin JounQin added bug Something isn't working enhancement New feature or request labels Dec 13, 2023
@JounQin JounQin self-assigned this Dec 13, 2023
Copy link

changeset-bot bot commented Dec 13, 2023

🦋 Changeset detected

Latest commit: d770029

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
domiso Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@JounQin JounQin force-pushed the test/dompurify_cases branch 17 times, most recently from ad98e27 to 2d89da9 Compare December 14, 2023 00:56
Copy link

codesandbox-ci bot commented Dec 14, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Copy link

codecov bot commented Dec 14, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (21ee166) 97.56% compared to head (d770029) 100.00%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main       #37      +/-   ##
===========================================
+ Coverage   97.56%   100.00%   +2.43%     
===========================================
  Files           1         1              
  Lines          41        96      +55     
  Branches       13        45      +32     
===========================================
+ Hits           40        96      +56     
+ Misses          1         0       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@JounQin JounQin force-pushed the test/dompurify_cases branch 2 times, most recently from a4d9c57 to 29efa24 Compare December 14, 2023 01:09
Copy link
Contributor

github-actions bot commented Dec 14, 2023

size-limit report 📦

Path Size
lib/index.js 895 B (+150.71% 🔺)

@JounQin JounQin force-pushed the test/dompurify_cases branch from 29efa24 to a37eec1 Compare December 14, 2023 01:12
@JounQin JounQin force-pushed the test/dompurify_cases branch from 67e32f6 to 51d0937 Compare December 14, 2023 15:13
src/index.ts Dismissed Show dismissed Hide dismissed
@JounQin JounQin force-pushed the test/dompurify_cases branch from 71a9af1 to d6816c1 Compare December 14, 2023 16:11
@JounQin JounQin force-pushed the test/dompurify_cases branch from d6816c1 to 6b444bc Compare December 14, 2023 16:16
@JounQin JounQin merged commit 462eac0 into main Dec 14, 2023
25 checks passed
@JounQin JounQin deleted the test/dompurify_cases branch December 14, 2023 16:52
@github-actions github-actions bot mentioned this pull request Dec 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants