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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs(purify): Update README.md #844

Merged
merged 3 commits into from
Aug 4, 2023
Merged

docs(purify): Update README.md #844

merged 3 commits into from
Aug 4, 2023

Conversation

ssi02014
Copy link
Contributor

@ssi02014 ssi02014 commented Aug 4, 2023

Summary

Hello 馃憢 @cure53
I think the current "README.md" is pretty good, but I've made some changes for readability and consistency.

In the case of "Can I configure DOMPurify?", all the example code is crammed into one code box, which I think is bad for readability.

I read the "README.md" and thought it could be improved, so I created a pull request.
Please review this pull request to see if it's appropriate. 馃檹

Tasks

  • Update README.md

馃摲 preview Image

釀夅叧釀忈叧釀呩叺釂剦釁a喓 2023-08-04 釀嬦叐釀掅叜 5 26 20 釀夅叧釀忈叧釀呩叺釂剦釁a喓 2023-08-04 釀嬦叐釀掅叜 5 26 41 釀夅叧釀忈叧釀呩叺釂剦釁a喓 2023-08-04 釀嬦叐釀掅叜 5 26 51

@ssi02014 ssi02014 changed the title Docs/purify docs(purify): Update README.md Aug 4, 2023
@cure53
Copy link
Owner

cure53 commented Aug 4, 2023

Looks great, thanks 馃槃

@cure53 cure53 merged commit 770cddf into cure53:main Aug 4, 2023
6 checks passed
@ssi02014 ssi02014 deleted the docs/purify branch August 4, 2023 13:12
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.

None yet

2 participants