Skip to content

test: improve test coverage SourceMap API - #36089

Merged
Trott merged 0 commit into
nodejs:masterfrom
juanarbol:juanarbol/improve-sourcemaps-coverage
Nov 24, 2020
Merged

test: improve test coverage SourceMap API#36089
Trott merged 0 commit into
nodejs:masterfrom
juanarbol:juanarbol/improve-sourcemaps-coverage

Conversation

@juanarbol

Copy link
Copy Markdown
Member
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines

@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Nov 11, 2020
@TrottTrott added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 14, 2020
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 14, 2020
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@juanarboljuanarbol added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 23, 2020
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 23, 2020
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@Trott

Copy link
Copy Markdown
Member

Landed in 68795f9

@TrottTrott closed this Nov 24, 2020
@Trott
Trottforce-pushed the juanarbol/improve-sourcemaps-coverage branch from aae4bbe to 68795f9CompareNovember 24, 2020 12:44
@Trott
Trott merged commit 68795f9 into nodejs:masterNov 24, 2020
danielleadams pushed a commit that referenced this pull request Dec 7, 2020
PR-URL: #36089
Reviewed-By: Zeyu Yang <himself65@outlook.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@danielleadamsdanielleadams mentioned this pull request Dec 7, 2020
@juanarbol
juanarbol deleted the juanarbol/improve-sourcemaps-coverage branch January 19, 2021 16:26
targos pushed a commit that referenced this pull request May 1, 2021
PR-URL: #36089
Reviewed-By: Zeyu Yang <himself65@outlook.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@danielleadamsdanielleadams mentioned this pull request May 3, 2021
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@juanarbol@nodejs-github-bot@Trott@himself65