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

Update the underlying build systems #71

Merged
merged 10 commits into from
Nov 9, 2021
Merged

Update the underlying build systems #71

merged 10 commits into from
Nov 9, 2021

Conversation

woksin
Copy link
Contributor

@woksin woksin commented Nov 8, 2021

Summary

Reducing the development foot print by using a more streamlined and slim version of our @doluttle/typescript.build package. The only controversial thing here is targeting ES2017 instead of ES2015, I don't think that this change is strictly necessary in order for this PR to work, but most it will reduce the size of the output and all modern browsers should support it.

Added

  • Extension methods for converting guids to and from protobuf

Changed

  • Target ES2017 instead of ES2015
  • Upgrade to newest version of @dolittle/typescript.build
  • Change package.json scripts to be more streamlined and using mocha, eslint and tsc for testing, linting and compiling the code instead of using gulp
  • Changed specs to use @dolittle/typescript.testing describeThis function that generates a more useful output when running the tests
  • Apply lint fix to fix lint errors

Fixed

  • Fixed some specs in @dolittle/sdk.protobuf
  • Fixes the wallaby configuration

@woksin woksin added the patch label Nov 8, 2021
woksin and others added 2 commits November 8, 2021 13:14
* Run yarn lint fix

* Fix ci script and add cleaning of tests

* Add peer dependencies
@woksin woksin merged commit f27745a into master Nov 9, 2021
@woksin woksin deleted the 19.0.1-newbuild branch November 9, 2021 13:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant