FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Comparing nodejs:main...MoLow:main · nodejs/node-core-test · GitHub

/ node-core-test Public
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: nodejs/node-core-test
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
Could not load branches
Nothing to show
{{ refName }}
...
head repository: MoLow/node-core-test
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: main
Choose a head ref
Could not load branches
Nothing to show
{{ refName }}
Checking mergeability… Don’t worry, you can still create the pull request.
  • 12 commits
  • 42 files changed
  • 6 contributors

Commits on Feb 7, 2023

  1. chore: use readable-stream

    MoLow committed Feb 7, 2023
    Configuration menu
    Copy the full SHA
    e123c03 View commit details
    Browse the repository at this point in the history

  2. feat: add reporters 

    PR-URL: nodejs/node#45712
Fixes: nodejs/node#45648
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
(cherry picked from commit a1b27b25bb01aadd3fd2714e4b136db11b7eb85a)
    MoLow committed Feb 7, 2023
    Configuration menu
    Copy the full SHA
    07a637e View commit details
    Browse the repository at this point in the history

  3. fix: move test reporter loading 

    Move the logic for handling --test-reporter out of the
general module loader and into the test_runner subsystem.

PR-URL: nodejs/node#45923
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jacob Smith <jacob@frende.me>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
(cherry picked from commit 12c0571c8fece32d274eaf0ae197c0eb1948fe11)
    GeoffreyBooth authored and MoLow committed Feb 7, 2023
    Configuration menu
    Copy the full SHA
    57de87d View commit details
    Browse the repository at this point in the history

  4. chore: use os.availableParallelism() 

    This commit addresses an existing TODO in the code by moving
to the new os.availableParallelism() instead of os.cpus().length.

PR-URL: nodejs/node#45969
Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
(cherry picked from commit fe5710effcb0b336853657e424da04407b0e49ad)
    cjihrig authored and MoLow committed Feb 7, 2023
    Configuration menu
    Copy the full SHA
    3399e8f View commit details
    Browse the repository at this point in the history

  5. feat: report file in test runner events 

    PR-URL: nodejs/node#46030
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
(cherry picked from commit 9eb363a3e00dbba572756c7ed314273f17ea8e2e)
    MoLow committed Feb 7, 2023
    Configuration menu
    Copy the full SHA
    29c73b4 View commit details
    Browse the repository at this point in the history

  6. fix: make built in reporters internal 

    This commit updates the test runner to make the built in test
reporters internal modules.

PR-URL: nodejs/node#46092
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
(cherry picked from commit 4788c0d2a02a2e7c7088c6f39d9f13a4209ba863)
    cjihrig authored and MoLow committed Feb 7, 2023
    Configuration menu
    Copy the full SHA
    e50df0e View commit details
    Browse the repository at this point in the history

  7. chore: update comment to comply with eslint no-fallthrough rule 

    PR-URL: nodejs/node#46258
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
(cherry picked from commit 931f035bac8326a11f42fc05463d5b21d9bec502)
    aduh95 authored and MoLow committed Feb 7, 2023
    Configuration menu
    Copy the full SHA
    3b42f3d View commit details
    Browse the repository at this point in the history

  8. test: fix tap escaping with and without --test 

    PR-URL: nodejs/node#46311
Fixes: nodejs/node#45836
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
(cherry picked from commit 2f38c74e263ed2e7f3b087efb9adee2442dd25c4)
    pulkit-30 authored and MoLow committed Feb 7, 2023
    Configuration menu
    Copy the full SHA
    dfca2fd View commit details
    Browse the repository at this point in the history

  9. fix: top-level diagnostics not ommited when running with --test 

    PR-URL: nodejs/node#46441
Fixes: nodejs/node#45910
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
(cherry picked from commit 61c65b066b098cf47f89206212864ec1cddb8782)
    pulkit-30 authored and MoLow committed Feb 7, 2023
    Configuration menu
    Copy the full SHA
    2434b16 View commit details
    Browse the repository at this point in the history

  10. fix: fix missing test diagnostics 

    PR-URL: nodejs/node#46450
Fixes: nodejs/node#45911
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
(cherry picked from commit 1118db718c8429f5f343aca90ccb570244e282b4)
    MoLow committed Feb 7, 2023
    Configuration menu
    Copy the full SHA
    22ccbdf View commit details
    Browse the repository at this point in the history

  11. fix: fix tap parser fails if a test logs a number 

    PR-URL: nodejs/node#46056
Fixes: nodejs/node#46048
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
(cherry picked from commit 4c08c20e575a0954fe3977a20e9f52b4980a2e48)
    pulkit-30 authored and MoLow committed Feb 7, 2023
    Configuration menu
    Copy the full SHA
    fba53c3 View commit details
    Browse the repository at this point in the history

  12. docs: update test concurrency description / default values 

    PR-URL: nodejs/node#46457
Reviewed-By: Moshe Atlow <moshe@atlow.co.il>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Jacob Smith <jacob@frende.me>
(cherry picked from commit 7d68b7bbfc9ffbd2ad0913972ac0b1a315679b06)
    richiemccoll authored and MoLow committed Feb 7, 2023
    Configuration menu
    Copy the full SHA
    a382788 View commit details
    Browse the repository at this point in the history
Loading
Back | FazBrowse Home | New Git URL