Skip to content
This repository has been archived by the owner on Feb 26, 2021. It is now read-only.

re-enable unit and full-app tests #172

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

noahsw
Copy link

@noahsw noahsw commented Aug 5, 2016

Tests were removed from CI in #164 but they're working fine for me on master so we should add them back.

Open question is whether we should be running meteor npm install or npm install, but let's see if CI passes.

@mrmowgli
Copy link

How did you get the file permissions to work on circlecl? I'm trying to update the tests in the coffeescript branch, and am running into this. I assume I should be modifying the pre section of the circle.yml?

@hwillson hwillson self-assigned this Sep 16, 2016
@hwillson
Copy link
Contributor

Hi @noahsw - thanks for the PR. Are you still working on this? Let me know if you are, or if you'd like me to take a look.

@noahsw
Copy link
Author

noahsw commented Sep 19, 2016

Hi @hwillson - sorry I ran into CI problems and had to take a break. Maybe this is why the tests were removed in the first place... I hope someone can figure it out though!

@hwillson
Copy link
Contributor

No problem @noahsw - I'll take a look. Thanks again!

@GeoffreyBooth
Copy link
Contributor

See also #210

@hwillson hwillson removed their assignment Apr 11, 2017
@CLAassistant
Copy link

CLAassistant commented Oct 7, 2019

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
0 out of 2 committers have signed the CLA.

❌ noahsw
❌ DominikGuzei
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants