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

docs: add Sentry's unplugin repo #294

Merged
merged 2 commits into from
Mar 22, 2023
Merged

docs: add Sentry's unplugin repo #294

merged 2 commits into from
Mar 22, 2023

Conversation

smeubank
Copy link
Contributor

@smeubank smeubank commented Mar 21, 2023

I know there is already a community repo from another user which is awesome! But wanted to call out Sentry has one it released late last year as well.

I know there is already a community repo from another user which is awesome! But wanted to call out Sentry has one it release late last year as well.
@Lms24
Copy link

Lms24 commented Mar 21, 2023

cc @kricsleo as you are the author of the other unplugin-sentry. If you want to collaborate with us on our official version, feel free to let us know! Happy to review PRs :)

README.md Outdated Show resolved Hide resolved
@kricsleo
Copy link
Contributor

cc @kricsleo as you are the author of the other unplugin-sentry. If you want to collaborate with us on our official version, feel free to let us know! Happy to review PRs :)

@Lms24 Actually, this issue existed: kricsleo/unplugin-sentry#3. It looks like we are doing things slightly differently, I think it's OK to leave both packages.

@Lms24
Copy link

Lms24 commented Mar 22, 2023

Sure, totally fine, thanks for the reply :) just took a closer look at your plugin and it's great to see that you added additional features. We probably need to spend some time on our plugin to catch up 😅

@antfu antfu changed the title Add Sentry's unplugin repo docs: add Sentry's unplugin repo Mar 22, 2023
@antfu antfu merged commit a2ebf99 into unjs:main Mar 22, 2023
@smeubank smeubank deleted the patch-1 branch March 22, 2023 13:47
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.

4 participants