Skip to content

Conversation

@mscdex
Copy link
Contributor

@mscdex mscdex commented Apr 22, 2016

Checklist
  • the commit message follows commit guidelines
Affected core subsystem(s)
  • build
Description of change

This commit excludes generated tap files from tarballs.

@mscdex mscdex added the build Issues and PRs related to build files or the CI. label Apr 22, 2016
@mscdex mscdex force-pushed the build-exclude-tap-from-tarballs branch from 6e97c92 to 389edbd Compare April 22, 2016 14:26
@jasnell
Copy link
Member

jasnell commented Apr 22, 2016

LGTM

1 similar comment
@JungMinu
Copy link
Member

LGTM

@jbergstroem
Copy link
Member

jbergstroem commented Apr 24, 2016

How about *.tap? Regardless LGTM.

Edit: rationale being we might have lint.tap, etc.

@Fishrock123
Copy link
Contributor

lgtm

@estliberitas estliberitas force-pushed the master branch 2 times, most recently from 7da4fd4 to c7066fb Compare April 26, 2016 05:23
PR-URL: nodejs#6348
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
@mscdex mscdex force-pushed the build-exclude-tap-from-tarballs branch from 389edbd to be54989 Compare April 28, 2016 15:28
@mscdex mscdex merged commit be54989 into nodejs:master Apr 28, 2016
@mscdex mscdex deleted the build-exclude-tap-from-tarballs branch April 28, 2016 15:34
Fishrock123 pushed a commit that referenced this pull request May 4, 2016
PR-URL: #6348
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Minwoo Jung <jmwsoft@gmail.com>
Reviewed-By: Johan Bergström <bugs@bergstroem.nu>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Issues and PRs related to build files or the CI.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants