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

Export TypeScript type definitions to local folder #3418

Merged
merged 6 commits into from
Jun 14, 2021
Merged

Conversation

schloerke
Copy link
Collaborator

Am able to import this package directly and use within VS Code:
Screen Shot 2021-06-11 at 12 56 49 PM

In package.json, by setting main to "", then we explicitly state we are not exporting any logic, only types. This also prompted the pkg name @types/shiny.

In a different PR, we should set the package version from the R version.

(Everything in src_d is auto generated)

@schloerke schloerke requested a review from nstrayer June 11, 2021 17:02
@schloerke
Copy link
Collaborator Author

Merging now so that we may play with the changes in other packages. Probably not the final form of the package shape.

@schloerke schloerke merged commit 3fe8c27 into master Jun 14, 2021
@schloerke schloerke deleted the ts_export branch June 14, 2021 18:25
schloerke added a commit that referenced this pull request Jun 15, 2021
* master:
  Move `./srcts` configs to top level to support types installation from GitHub (#3425)
  insertTab() now handles position correctly when target is NULL (#3404)
  yarn add node-gyp; yarn build (#3424)
  Export TypeScript type definitions to local folder (#3418)
  TypeScript: Remove `any` types / improve type definitions (#3414)
  Better color constrasting in sliderInput() (#3366)
  Use ggplot2::get_alt_text() if available to provide better default alt text (#3398)
  Set selectize dropdownParent to "body" to prevent clipping
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.

1 participant