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

Added typescript plugins dev guide #497

Merged
merged 4 commits into from
Feb 24, 2025
Merged

Added typescript plugins dev guide #497

merged 4 commits into from
Feb 24, 2025

Conversation

mrigankmg
Copy link
Contributor

@mrigankmg mrigankmg commented Aug 29, 2024

Added dev guide for typescript plugins.

Change Type (required)

Indicate the type of change your pull request is:

  • patch
  • minor
  • major

Does your PR have any documentation updates?

  • Updated docs
  • No Update needed
  • Unable to update docs

@mrigankmg mrigankmg requested a review from adierkens as a code owner August 29, 2024 17:01
@mrigankmg mrigankmg changed the title Added typescript plugins dev guide. Added typescript plugins dev guide Aug 29, 2024
@mrigankmg mrigankmg requested a review from hborawski August 29, 2024 17:01
@hborawski hborawski requested a review from cehan-Chloe August 29, 2024 17:17
@mrigankmg mrigankmg force-pushed the typescript-dev-guide branch 2 times, most recently from a2a55f9 to d89a41c Compare August 29, 2024 17:30
@mrigankmg mrigankmg force-pushed the typescript-dev-guide branch from d89a41c to 0463a4f Compare August 29, 2024 17:44
{
"name": "@player-ui/example-plugin",
"version": "0.0.0-PLACEHOLDER",
"main": "src/index.tsx",
Copy link
Member

Choose a reason for hiding this comment

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

Nitpick but technically the entrypoint to a React Plugin doesn't have to be a .tsx file

@mrigankmg mrigankmg force-pushed the typescript-dev-guide branch from 4b490cf to 2c4be0f Compare August 30, 2024 00:14
Copy link

codecov bot commented Aug 30, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.97%. Comparing base (9d3589e) to head (e409fe2).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #497   +/-   ##
=======================================
  Coverage   91.97%   91.97%           
=======================================
  Files         340      340           
  Lines       26960    26960           
  Branches     1945     1945           
=======================================
  Hits        24796    24796           
  Misses       2150     2150           
  Partials       14       14           

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

@mrigankmg mrigankmg force-pushed the typescript-dev-guide branch from 2c4be0f to e409fe2 Compare August 30, 2024 13:56
@KetanReddy KetanReddy added the patch Increment the patch version when merged label Feb 24, 2025
@KetanReddy KetanReddy enabled auto-merge February 24, 2025 18:46
@KetanReddy KetanReddy merged commit 9906f93 into main Feb 24, 2025
3 of 7 checks passed
@KetanReddy KetanReddy deleted the typescript-dev-guide branch February 24, 2025 18:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch version when merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants