10000 Update README.md solves issue #50 by soundslikeinfo · Pull Request #54 · kepano/defuddle · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Update README.md solves issue #50 #54

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

Merged
merged 1 commit into from
May 28, 2025
Merged

Conversation

soundslikeinfo
Copy link
Contributor

Update README with a working version for Defuddle with options

Update README with a working version for Defuddle with options
@soundslikeinfo soundslikeinfo changed the title Update README.md solves #50 Update README.md solves issue #50 May 25, 2025
@soundslikeinfo
Copy link
Contributor Author

Details about the options would also need to be adjusted, since url is moved outside the options object.

@kepano kepano merged commit 6c17079 into kepano:main May 28, 2025
Daniel-u13 added a commit to Daniel-u13/defuddle that referenced this pull request Jun 1, 2025
commit 6c17079
Author: Sounds Like Info <gkerstine@me.com>
Date:   Wed May 28 08:25:55 2025 -0700

    Update README.md (kepano#54)

    Update README with a working version for Defuddle with options

commit 928c11a
Author: Jeremy Morrell <morrell.jeremy@gmail.com>
Date:   Wed May 28 08:25:25 2025 -0700

    Fix readme typo (kepano#55)

    https://github.com/kepano/defuddle/blob/cb4291db0f24cac0d0674d9e35fc0089338da2da/src/index.ts#L5

    The example code here for the browser is slightly off. `Defuddle` is exported as default, which means we can import it directly.

    It used to be correct, but was overlooked when adding the Node example code here: kepano#25
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.

3 participants
0