Skip to content

Update build setting#7

Merged
changchanghwang merged 2 commits intomainfrom
build-setting
Aug 1, 2023
Merged

Update build setting#7
changchanghwang merged 2 commits intomainfrom
build-setting

Conversation

@ActaV-N
Copy link
Copy Markdown
Collaborator

@ActaV-N ActaV-N commented Jul 31, 2023 •
edited
Loading

Description of change

  • commonjs is default setting of module in tsconfig → so, extra build script's outDir changed to dist/esm from dist/cjs
  • added tsconfig.build.json so that test files can be excluded from package

Pull-Request Checklist

  • Code is up-to-date with the main branch
  • There are new or updated unit tests validating the change
  • Documentation has been updated to reflect this change
  • The new commits follow conventions explained in CONTRIBUTING.md

@ActaV-N ActaV-N requested a review from changchanghwang July 31, 2023 15:50
@changchanghwang changchanghwang merged commit 133b057 into main Aug 1, 2023
@changchanghwang changchanghwang deleted the build-setting branch August 1, 2023 00:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@changchanghwang changchanghwang changchanghwang approved these changes

Assignees

@changchanghwang changchanghwang

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ActaV-N @changchanghwang