Skip to content
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

moved it tests to a separate dir, and separated test tasks in grunt #3

Merged
merged 2 commits into from
Feb 26, 2015

Conversation

natoverse
Copy link
Contributor

Getting the builds a little more robust and flexible.

@natoverse
Copy link
Contributor Author

Yes, for sure. I was really just making some minor edits to fix the badge formatting, and need to add quite a bit more actual content to the readme.

@darthtrevino
Copy link
Contributor

Travis is sure taking it's sweet time. It doesn't look like it's started yet. I wonder if they have some kind of throttle in place for free tiers.

@natoverse
Copy link
Contributor Author

I think so. The first pull request build took almost 30 minutes. However, once it was done it reported the actual build time as 32 seconds. So I think they are putting in some wait time to reduce possible abuse.

@darthtrevino
Copy link
Contributor

I would wager that it's a lot faster with the paid tier then. Only $129/mo

@natoverse
Copy link
Contributor Author

I don't see what the conflicts are that prevent auto-merge.

@davija
Copy link
Member

davija commented Feb 26, 2015

Can't be automatically merged due to README.md file being changed on master & this branch.

@natoverse
Copy link
Contributor Author

Hmm, I didn't realize I changed it on master after branching. Can you resolve the commit and merge?

@davija davija merged commit a29c21c into master Feb 26, 2015
davija pushed a commit that referenced this pull request Feb 26, 2015
Merging changes from pull request #3
@natoverse natoverse deleted the rejiggering-tests branch February 26, 2015 00:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants