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(core): add exclude option to enqueueLinks #1786

Merged
merged 11 commits into from
Mar 9, 2023
Merged

Conversation

jackHedaya
Copy link
Contributor

@jackHedaya jackHedaya commented Feb 12, 2023

This is a PR to implement the functionality described in #1785. Closes #1785.

Copy link
Member

@B4nan B4nan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, looking good!

packages/core/src/enqueue_links/enqueue_links.ts Outdated Show resolved Hide resolved
packages/core/src/enqueue_links/enqueue_links.ts Outdated Show resolved Hide resolved
test/core/enqueue_links/enqueue_links.test.ts Outdated Show resolved Hide resolved
packages/core/src/enqueue_links/enqueue_links.ts Outdated Show resolved Hide resolved
packages/core/src/enqueue_links/shared.ts Outdated Show resolved Hide resolved
packages/core/src/enqueue_links/shared.ts Outdated Show resolved Hide resolved
@jackHedaya jackHedaya requested review from vladfrangu and B4nan and removed request for vladfrangu February 13, 2023 16:34
@B4nan B4nan changed the title feat(core): Add blacklist option feat(core): add exluce option to enqueueLinks Feb 13, 2023
@B4nan B4nan changed the title feat(core): add exluce option to enqueueLinks feat(core): add exlude option to enqueueLinks Feb 13, 2023
@B4nan B4nan changed the title feat(core): add exlude option to enqueueLinks feat(core): add exclude option to enqueueLinks Feb 13, 2023
@jackHedaya
Copy link
Contributor Author

@vladfrangu Friendly bump 🙂

@B4nan B4nan merged commit 2e833dc into apify:master Mar 9, 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.

Blacklist for enqueueLinks
3 participants