Skip to content
This repository was archived by the owner on Aug 23, 2019. It is now read-only.

refactor: use aegir#5

Merged
daviddias merged 1 commit intomasterfrom
aegir
Apr 15, 2016
Merged

refactor: use aegir#5
daviddias merged 1 commit intomasterfrom
aegir

Conversation

@dignifiedquire
Copy link
Copy Markdown
Member

@diasdavid can't get the gulp file to do the correct setup, could you take a look please?

@dignifiedquire dignifiedquire mentioned this pull request Apr 15, 2016
36 tasks
@daviddias
Copy link
Copy Markdown
Member

Why create a gulp file for extra setup? The test is quite small and only runs in Node.js

@dignifiedquire
Copy link
Copy Markdown
Member Author

To have it run in the browser as well, or is this intended only for node usage?

@daviddias
Copy link
Copy Markdown
Member

The browser checks were not made yet. Let's not urge that, the code might need refactoring anyway.

@dignifiedquire
Copy link
Copy Markdown
Member Author

okay will move things to node only

@dignifiedquire
Copy link
Copy Markdown
Member Author

@diasdavid done

@coveralls
Copy link
Copy Markdown

Coverage Status

Changes Unknown when pulling 8c02c85 on aegir into * on master*.

@daviddias
Copy link
Copy Markdown
Member

woooo :D

@daviddias daviddias merged commit aa06155 into master Apr 15, 2016
@daviddias daviddias deleted the aegir branch April 15, 2016 22:40
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants