Skip to content

Add tests and update build scripts#195

Merged
compulim merged 40 commits intomicrosoft:masterfrom
compulim:feat-tests
Sep 13, 2019
Merged

Add tests and update build scripts#195
compulim merged 40 commits intomicrosoft:masterfrom
compulim:feat-tests

Conversation

@compulim
Copy link
Collaborator

@compulim compulim commented Sep 11, 2019

Description

  • Added new test harness that integrate with a production bot
  • Updated build scripts
  • Bumped all dependencies

Changelog

Breaking Changes

  • Build folders updated
    • /dist/ contains JavaScript bundle
      • /dist/directline.js is now in lowercase
    • /lib/ contains ES modules and type definitions
  • Build scripts updated
    • npm run build: Development build, with instrumentation code, one-off
    • npm run start: Development build, with instrumentation code, with watch
    • npm run prepublishOnly: Production build, minified, one-off

Changed

Copy link
Contributor

@corinagum corinagum left a comment

Choose a reason for hiding this comment

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

LGTM. Left some comments but @tdurnford should also take a look :)

Copy link
Contributor

@tdurnford tdurnford left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@compulim
Copy link
Collaborator Author

Phew~~~

@compulim
Copy link
Collaborator Author

Survived another brutal round of PR review. 🥇

@compulim
Copy link
Collaborator Author

I am going to change one more thing, on the promiseRaceMap. Prefer to have a single stack rather than 3 stacks.

@compulim
Copy link
Collaborator Author

compulim commented Sep 12, 2019

And stupid me why it is a stack (push/pop) but not a queue (push/shift). 😵

It should be a queue.

@compulim
Copy link
Collaborator Author

🔔 another round.

Copy link
Contributor

@tdurnford tdurnford left a comment

Choose a reason for hiding this comment

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

LGTM

@compulim compulim merged commit 048651f into microsoft:master Sep 13, 2019
@compulim compulim deleted the feat-tests branch September 13, 2019 21:29
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.

3 participants