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

Allow text-based file objects for HTML and CSS classes #2022

Merged

Conversation

ViktorShev
Copy link
Contributor

@ViktorShev ViktorShev commented Dec 14, 2023

Related to: #2023

@liZe
Copy link
Member

liZe commented Dec 14, 2023

Hi!

Thanks for this pull request. Could you please explain which problem you’re trying to solve?

@ViktorShev
Copy link
Contributor Author

ViktorShev commented Dec 14, 2023

@liZe Yes! just finished filing an issue.

@liZe
Copy link
Member

liZe commented Dec 16, 2023

Thanks a lot for this PR and the related issue! I’ve added tests, and allowed text-based file objects for CSS class too. Feedback is welcome!

@liZe liZe added this to the 61.0 milestone Dec 16, 2023
@liZe liZe merged commit 05390c8 into Kozea:main Dec 16, 2023
6 checks passed
@liZe liZe changed the title Filter args with value None before calling html5lib.parse Allow text-based file objects for HTML and CSS classes Dec 16, 2023
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