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
node 0.8 #3
Comments
We absolutely are. The Travis CI needs to be fixed. May have wanted to ask before breaking the build (or looking at the other repositories). |
alright. let me know how to proceed. i could fix this on travis but not sure how this would differ on appveyor |
Also, why was the version bumped patch instead of minor? The dependency update is providing a new feature, which is directly visible in that module's interface, so it should be a minor version bump to represent that there is a new feature available. |
Ignoring the questionable adherence to semver on what was published, it should at least be reasonable to wait for all-green the the CIs on the commit that is going to be published before publishing. I'm rebasing your changes on my commits from earlier this year that resolves the CI configurations for the istanbul module misbehaving. |
Builds should be resolved in |
Oh, and I guess on the Node.js 0.8 topic over-all, yes, Node.js 0.8 can be removed from the testing, but the module will need to undergo a major version bump at the same time to be fair to the users who are depending on the modules working. |
we still supporting this @dougwilson? https://github.com/pillarjs/resolve-path is starting to fail on 0.8. i think we can ditch it now...
The text was updated successfully, but these errors were encountered: