Skip to content

Get benchmark data to visualizer#878

Merged
Perryvw merged 14 commits intomasterfrom
benchmark-data-to-visualizer
Oct 21, 2020
Merged

Get benchmark data to visualizer#878
Perryvw merged 14 commits intomasterfrom
benchmark-data-to-visualizer

Conversation

@Perryvw
Copy link
Copy Markdown
Member

@Perryvw Perryvw commented May 18, 2020

No description provided.

package.json Outdated
"ts-jest": "^25.2.1",
"ts-node": "^8.6.2"
"ts-node": "^8.6.2",
"zlib": "^1.0.5"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

zlib is a builtin module, it doesn't need a dependency

@Perryvw Perryvw marked this pull request as ready for review June 10, 2020 16:01
@@ -62,7 +62,7 @@ jobs:
# NPM
- name: NPM master
# TODO Lua types is only added manually to test the benchmark PR this can be removed again once the PR is merged
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

remove comment aswell

@Perryvw Perryvw merged commit dae7aba into master Oct 21, 2020
@Perryvw Perryvw deleted the benchmark-data-to-visualizer branch October 21, 2020 18:01
lolleko added a commit to lolleko/TypeScriptToLua that referenced this pull request Nov 5, 2020
Benchmarks now hopefully work for pull requests from a fork
Fixed comparison info not beeing emmitted correctly after TypeScriptToLua#878
Bump actions/github-script from 0.9.0 to v3
Moved create benchmark check script to standalone js file
Perryvw pushed a commit that referenced this pull request Nov 23, 2020
* Fixed various benchmark issues

Benchmarks now hopefully work for pull requests from a fork
Fixed comparison info not beeing emmitted correctly after #878
Bump actions/github-script from 0.9.0 to v3
Moved create benchmark check script to standalone js file

* Formatting & Adjusted script path

* Pass core to the js file

* Output markdown directly to CLI instead of creating an actions check

* Fixed github-script usage

* Fixed visualizer data passing

* Linting :)

* Return benchmark info from github-script

* Fixed glow flags

* Update name of final step

* Update .github/workflows/ci.yml

Co-authored-by: ark120202 <ark120202@gmail.com>

Co-authored-by: ark120202 <ark120202@gmail.com>
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