Skip to content

Commit

Permalink
Get version done
Browse files Browse the repository at this point in the history
  • Loading branch information
orta committed Oct 29, 2016
1 parent e0b1467 commit 8cc6e49
Show file tree
Hide file tree
Showing 5 changed files with 18 additions and 5 deletions.
4 changes: 1 addition & 3 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,7 @@ Danger on Node, wonder what's going on? see [VISION.md](VISION.md)

### Get started?

This is like, kinda early. If you can take a bit of heat, it's usable in production as of 0.0.4.

Bah, there's something wrong with the deploy. So for now, it's not working as an imported module. [See this PR](https://github.com/artsy/emission/pull/385).
This is like, kinda early. If you can take a bit of heat, it's usable in production as of 0.0.10. Note: There is basically no error reporting ATM.

### Early Adopters

Expand Down
4 changes: 4 additions & 0 deletions changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

// Add your own contribution below

### 0.0.5-0.0.10

* Changes some files casing, added some logs, a bit of error reporting, and verifying everything works through npm - orta

### 0.0.4

* Danger edit an existing post, and delete it when it's not relevant - orta
Expand Down
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
{
"name": "danger",
"version": "0.0.6",
"version": "0.0.10",
"description": "Automate your culture",
"main": "distribution/danger.js",
"bin": {
Expand Down
10 changes: 9 additions & 1 deletion source/runner/Dangerfile.js
Original file line number Diff line number Diff line change
Expand Up @@ -50,10 +50,18 @@ export default class Dangerfile {
const context: any = {
fail,
console,
require,
danger: this.dsl
}

script.runInNewContext(context)
console.log("Running Script")
try {
script.runInNewContext(context)
}
catch (e) {
console.log(e.toString())
}

return results
}

Expand Down
3 changes: 3 additions & 0 deletions source/runner/Executor.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,12 @@ export default class Executor {
async run() {
const git = await this.platform.getReviewDiff()
const pr = await this.platform.getReviewInfo()
console.log("Got github deets")

const dsl = new DangerDSL(pr, git)
const dangerfile = new Dangerfile(dsl)
const results = await dangerfile.run("dangerfile.js")
console.log("Got results")

if (results.fails.length) {
process.exitCode = 1
Expand Down

0 comments on commit 8cc6e49

Please sign in to comment.