Skip to content

refactor(api): generate types next to modules#8391

Merged
lucasfernog merged 3 commits into
devfrom
refactor/api/types-next-to-modules-dev
Dec 14, 2023
Merged

refactor(api): generate types next to modules#8391
lucasfernog merged 3 commits into
devfrom
refactor/api/types-next-to-modules-dev

Conversation

@amrbashir
Copy link
Copy Markdown
Member

@amrbashir amrbashir commented Dec 13, 2023

What kind of change does this PR introduce?

  • Bugfix
  • Feature
  • Docs
  • New Binding issue #___
  • Code style update
  • Refactor
  • Build-related changes
  • Other, please describe:

Does this PR introduce a breaking change?

  • Yes, and the changes were approved in issue #___
  • No

Checklist

  • When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
  • A change file is added if any packages will require a version bump due to this PR per the instructions in the readme.
  • I have added a convincing reason for adding this feature, if necessary

Other information

this fixes an issue with `moduleResolution: node`
@amrbashir amrbashir requested a review from a team as a code owner December 13, 2023 17:52
@amrbashir amrbashir changed the title refactor(api): generate types next to modules" refactor(api): generate types next to modules Dec 13, 2023
Copy link
Copy Markdown
Member

@lucasfernog lucasfernog left a comment

Choose a reason for hiding this comment

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

Don't we need a change file here?

@amrbashir
Copy link
Copy Markdown
Member Author

amrbashir commented Dec 14, 2023

I mean we could but it is kinda of an internal change, should I still add it?
Edit: nvm I forgot it fixes an issue with moduelResoultion: node

@amrbashir
Copy link
Copy Markdown
Member Author

amrbashir commented Dec 14, 2023

change file added

@lucasfernog lucasfernog merged commit ef21b68 into dev Dec 14, 2023
@lucasfernog lucasfernog deleted the refactor/api/types-next-to-modules-dev branch December 14, 2023 16:27
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.

2 participants