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

Cheerio type fix #591

Merged
merged 4 commits into from
Aug 16, 2024
Merged

Cheerio type fix #591

merged 4 commits into from
Aug 16, 2024

Conversation

UNIDY2002
Copy link
Member

🎉 Thanks for taking time to contribute! 🎉

Before we move on, fill out the template so that we can better understand your contribution.

What is the purpose of this PR?

  • Fixing a bug
  • Adding a feature

Is there a related issue?

cheeriojs/cheerio#3974

  • I understand that adding a new feature before creating a feature request issue has a high chance of rejection.

Please describe your changes.

The project used "@types/cheerio": "0.22.35", but it does not match cheerio's version used (1.0.0-rc). When removing "@types/cheerio", lots of typing errors occurs.

This PR fixed those typing errors, as well as removed the deprecated usage of cheerio's default export.

Are there any possible drawbacks or side-effects?

How to verify the changes?

@UNIDY2002 UNIDY2002 merged commit b092e73 into master Aug 16, 2024
7 checks passed
@UNIDY2002 UNIDY2002 deleted the cheerio-type-fix branch August 16, 2024 04:11
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.

1 participant