Skip to content

fix(types): exposed types from overlay#222

Merged
Lms24 merged 3 commits into
getsentry:mainfrom
Shubhdeep12:fix/exports-types-overlay
Dec 5, 2023
Merged

fix(types): exposed types from overlay#222
Lms24 merged 3 commits into
getsentry:mainfrom
Shubhdeep12:fix/exports-types-overlay

Conversation

@Shubhdeep12
Copy link
Copy Markdown
Collaborator

Fixes: #219
Before opening this PR:

  • I added a Changeset Entry with pnpm changeset:add
  • I referenced issues that this PR addresses

@vercel
Copy link
Copy Markdown

vercel Bot commented Dec 5, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
spotlightjs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 5, 2023 9:10am

@codecov
Copy link
Copy Markdown

codecov Bot commented Dec 5, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (c538cd0) 32.47% compared to head (6ab6f4d) 32.47%.
Report is 1 commits behind head on main.

❗ Current head 6ab6f4d differs from pull request most recent head ab6d7bc. Consider uploading reports for the commit ab6d7bc to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #222   +/-   ##
=======================================
  Coverage   32.47%   32.47%           
=======================================
  Files          39       39           
  Lines        1869     1869           
  Branches       71       71           
=======================================
  Hits          607      607           
  Misses       1262     1262           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Shubhdeep12 Shubhdeep12 marked this pull request as ready for review December 5, 2023 07:50
Copy link
Copy Markdown
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

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

Thanks for fixing! Two small comments otherwise ready to go!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

let's also set the top level types property in this file

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Done @Lms24. Please check.

Comment thread .changeset/gold-numbers-remain.md Outdated
Co-authored-by: Lukas Stracke <lukas.stracke@sentry.io>
Copy link
Copy Markdown
Member

@Lms24 Lms24 left a comment

Choose a reason for hiding this comment

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

Thanks!

@Lms24 Lms24 merged commit 7f5b581 into getsentry:main Dec 5, 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.

Can't add spotlight - Could not find a declaration file for module '@spotlightjs/spotlight'.

2 participants