/core

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

Ignore tests/raw-test.js generated for tests #508

Closed
wants to merge 1 commit into
base: master
from

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@jonathanperret @evancz
@jonathanperret
Contributor

jonathanperret commented Feb 25, 2016

Following the merge of #447, a file tests/raw-test.js is generated during a test run. It should be ignored by Git.
@jonathanperret
Ignore tests/raw-test.js generated for tests 
Following the merge of #447, a file `tests/raw-test.js` is generated
during a test run. It should be ignored by Git.
4469726
@evancz

This comment has been minimized.

Show comment
Hide comment
@evancz

evancz May 19, 2016

Member

I don't think the testing pipeline works this way anymore. @eeue56 may know better. I am going to close assuming I am correct. Let's open a new one if I am mistaken.
Member

evancz commented May 19, 2016

I don't think the testing pipeline works this way anymore. @eeue56 may know better. I am going to close assuming I am correct. Let's open a new one if I am mistaken.

@evancz evancz closed this May 19, 2016

@jonathanperret

This comment has been minimized.

Show comment
Hide comment
@jonathanperret

jonathanperret May 20, 2016

Contributor

You're right, 600e1d1 removed raw-test.js. Good catch.
Contributor

jonathanperret commented May 20, 2016

You're right, 600e1d1 removed raw-test.js. Good catch.

@jonathanperret jonathanperret deleted the jonathanperret:ignore-raw-test branch May 20, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment