Skip to content

Bump dependencies #6

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

Merged
merged 3 commits into from
Oct 23, 2015
Merged

Conversation

erikkemperman
Copy link
Member

As threatened in #40.

@@ -1,6 +1,7 @@
language: node_js
node_js:
- '0.10'
- '0.12'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should add '4' also

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@phated Do we want to specify '0.10', '0.12', '4' throughout the related repositories?

And as I mentioned elsewhere it seems that some repositories, such as this one, are not triggering a travis build despite having a .travis.yml. Any idea why that is?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, I want all the projects to be tested on 0.10, 0.12 and 4

I'm not actually sure why travis didn't run for this PR or some of the others. I don't see any options related to that.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@phated Ok, I hope I'll have some time later this week to migrate some repos to eslint and update their .travis.yml while I'm at it. Is this the .eslintrc you want throughout?

For what it's worth, my fork does trigger a travis rebuild on commits, link. So the config is fine. Maybe you want to take a look at the settings on travis-ci.org for the main repo, I suspect that something needs doing there? Even just removing and re-adding the repo might help.

phated added a commit that referenced this pull request Oct 23, 2015
@phated phated merged commit 99c5e17 into gulpjs:master Oct 23, 2015
@erikkemperman erikkemperman deleted the feature-bump-dependencies branch October 27, 2015 13:00
phated added a commit that referenced this pull request Jun 25, 2016
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.

2 participants