Skip to content

Commit

Permalink
Merge pull request #2 from nknapp/fix-node-8
Browse files Browse the repository at this point in the history
Don't ignore trace and clarify for node 8
  • Loading branch information
nknapp committed Aug 25, 2017
2 parents 8ed02b6 + 431271c commit b419429
Show file tree
Hide file tree
Showing 9 changed files with 2,520 additions and 34 deletions.
2 changes: 2 additions & 0 deletions .eslintignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,2 @@
node_modules
coverage
3 changes: 3 additions & 0 deletions .eslintrc.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
module.exports = {
"extends": "standard"
};
16 changes: 5 additions & 11 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,19 +1,13 @@
sudo: false
language: "node_js"
node_js:
- "0.10"
- "0.12"
- "iojs"
- "4"
- "5"
- "6"
- "7"
before_script:
- npm install standard
- standard
- "8"
script:
- npm install istanbul
- istanbul cover ./node_modules/.bin/_mocha --report lcovonly
- npm install
- npm run coverage
- eslint --max-warnings 0 .
after_script:
- npm install coveralls
- cat ./coverage/lcov.info | coveralls
- cat ./coverage/lcov.info | coveralls
7 changes: 7 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,13 @@
# Release notes for `trace-and-clarify-if-possible`

<a name="current-release"></a>
# Version 1.0.1 (Thu, 24 Aug 2017 20:58:17 GMT)

* [608f7b4](https://github.com/nknapp/trace-and-clarify-if-possible/commit/608f7b4) Disable "trace@2" for node 8 - Nils Knappmeier
* [8639e6c](https://github.com/nknapp/trace-and-clarify-if-possible/commit/8639e6c) Chore setup - Nils Knappmeier
* [8ed02b6](https://github.com/nknapp/trace-and-clarify-if-possible/commit/8ed02b6) chore(package): update dependencies - greenkeeper[bot]


# Version 1.0.0 (Mon, 19 Dec 2016 21:49:01 GMT)

* [27ee212](https://github.com/nknapp/trace-and-clarify-if-possible/commit/27ee212) Initial implementation - Nils Knappmeier
Expand Down
2 changes: 1 addition & 1 deletion CONTRIBUTING.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
## Contributing
# Contributing

Contributions and feedback are always welcome. The expected procedure is the following:

Expand Down
13 changes: 7 additions & 6 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,10 +1,9 @@
# trace-and-clarify-if-possible

[![NPM version](https://badge.fury.io/js/trace-and-clarify-if-possible.svg)](http://badge.fury.io/js/trace-and-clarify-if-possible)
[![NPM version](https://img.shields.io/npm/v/trace-and-clarify-if-possible.svg)](https://npmjs.com/package/trace-and-clarify-if-possible)
[![Travis Build Status](https://travis-ci.org/nknapp/trace-and-clarify-if-possible.svg?branch=master)](https://travis-ci.org/nknapp/trace-and-clarify-if-possible)
[![Coverage Status](https://img.shields.io/coveralls/nknapp/trace-and-clarify-if-possible.svg)](https://coveralls.io/r/nknapp/trace-and-clarify-if-possible)


> Use `trace` and `clarify` if supported by the current node version
[trace](https://npmjs.com/package/trace) and [clarify](https://npmjs.com/package/clarify) can help greatly to analyze stack-traces
Expand Down Expand Up @@ -38,15 +37,17 @@ Or, you can put `--require trace-and-clarify-if-possible into your `test/mocha.o



## License
# License

`trace-and-clarify-if-possible` is published under the MIT-license.

`trace-and-clarify-if-possible` is published under the MIT-license.
See [LICENSE.md](LICENSE.md) for details.

## Release-Notes

# Release-Notes

For release notes, see [CHANGELOG.md](CHANGELOG.md)

## Contributing guidelines
# Contributing guidelines

See [CONTRIBUTING.md](CONTRIBUTING.md).
3 changes: 1 addition & 2 deletions index.js
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,8 @@

var versionStr = process.version.match(/v(\d+).*/)[1]
var majorVersion = Number(versionStr)
if (majorVersion >= 4) {
if (majorVersion >= 4 && majorVersion < 8) {
require('trace')
require('clarify')
console.log('Trace and clarify activated')
}

0 comments on commit b419429

Please sign in to comment.