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

more timings #578

Merged
merged 4 commits into from
Jun 12, 2019
Merged

more timings #578

merged 4 commits into from
Jun 12, 2019

Conversation

yoshuawuyts
Copy link
Member

Adds a whole bunch more timings to Choo; allows for better profiling of startup time.

Breaking change b/c choo.route and choo.router timings are swapped.

* master: (71 commits)
  6.13.3
  Revert init order change
  Add browser tests (#696)
  6.13.2
  Fix location missing on store init (#695)
  Typo Fix in Readme (#693)
  6.13.1
  Fix wrong this usage in nanohref integration (#689)
  Some spelling & typo fixes in readme (#688)
  Fix inspect script (#654)
  Add v6 tests (#674)
  Remove references to bel (#678)
  6.13.0
  Add hash option (#667)
  Add documentation on components (#673)
  Update README.md (#669)
  6.12.1
  6.12.0
  typings: add app to app.use cb (#665)
  update dependencies (#663)
  ...
@tornqvist tornqvist mentioned this pull request Jun 8, 2019
6 tasks
@tornqvist

This comment has been minimized.

* master:
  Update nanorouter (#701)
  Use Object.assign instead of xtend (#616)
Copy link
Collaborator

@bcomnes bcomnes left a comment

Choose a reason for hiding this comment

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

LGTM, are we still supporting timings since the timing API was screwed up?

@tornqvist
Copy link
Member

You mean in Node? That has sorted itself out for me in later versions of Node.

@bcomnes
Copy link
Collaborator

bcomnes commented Jun 12, 2019

I just heard something about timings api breaking, but yeah I figured it would be sorted out. yeah this is fine

@tornqvist tornqvist merged commit b373c7e into master Jun 12, 2019
@delete-merged-branch delete-merged-branch bot deleted the better-timings branch June 12, 2019 11:41
tornqvist added a commit to tornqvist/choo that referenced this pull request Jun 12, 2019
* master:
  Match route before init stores (choojs#698)
  more timings (choojs#578)
  Disable hash routing by default (choojs#699)
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.

3 participants