Skip to content

refactor tsconfig -- make consistent, fix sourcemap warning in core#547

Merged
silesky merged 3 commits intomasterfrom
chore/refactor-tsconfig-to-better-separate-build
Aug 2, 2022
Merged

refactor tsconfig -- make consistent, fix sourcemap warning in core#547
silesky merged 3 commits intomasterfrom
chore/refactor-tsconfig-to-better-separate-build

Conversation

@silesky
Copy link
Copy Markdown
Contributor

@silesky silesky commented Jul 18, 2022

  • generate a types folder in browser, similar to the rest of the packages.

  • reference tsconfig.build.json in webpack, which is the correct tsconfig to reference.

  • include src and declaration maps to improve IDE experience.

  • fix sourcemap warning in core.

  • make tsconfig and tsconfig.build.json consistent across packages -- tsconfig.build.json will be a build configuration; therefore, it has an outDir and typesDir and be responsible for specifics around generating the dist folder. tsconfig.json is a base tsconfig and should contains information around the overall environment (e.g. target, libs), with enough info run lint / jest / etc correctly without building a dist folder or generating declarations.

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Jul 18, 2022

🦋 Changeset detected

Latest commit: b5c46ff

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@segment/analytics-core Patch
@segment/analytics-next Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@silesky silesky force-pushed the chore/refactor-tsconfig-to-better-separate-build branch from ec25e44 to 931660d Compare July 18, 2022 03:12
@silesky silesky requested review from chrisradek, pooyaj and zikaari July 18, 2022 13:35
Copy link
Copy Markdown
Contributor

@zikaari zikaari left a comment

Choose a reason for hiding this comment

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

Nice!

@silesky silesky force-pushed the chore/refactor-tsconfig-to-better-separate-build branch from 931660d to f8cae58 Compare August 2, 2022 22:02
@silesky silesky force-pushed the chore/refactor-tsconfig-to-better-separate-build branch from 52f4dae to 603d944 Compare August 2, 2022 22:34
@silesky silesky merged commit 93c8f1f into master Aug 2, 2022
@silesky silesky deleted the chore/refactor-tsconfig-to-better-separate-build branch August 2, 2022 23:06
@github-actions github-actions bot mentioned this pull request Aug 2, 2022
@silesky silesky changed the title refactor tsconfig -- make consistent refactor tsconfig -- make consistent, fix sourcemap warning in core Aug 2, 2022
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