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

Update settings.json #1

Closed
wants to merge 1 commit into from
Closed

Update settings.json #1

wants to merge 1 commit into from

Conversation

flovilmart
Copy link
Contributor

No description provided.

@orta
Copy link

orta commented Jul 1, 2017

The duplicate messages is a tad worrying

@flovilmart
Copy link
Contributor Author

effectively, I can try to pull the heroku logs

@flovilmart
Copy link
Contributor Author

@orta the danger file effectively ran twice:

2017-07-01T19:38:01.279445+00:00 app[web.1]: info: [router] -  - passing to Dangerfile rule router
2017-07-01T19:38:01.280554+00:00 app[web.1]: info: [runner] - Event Settings: {
2017-07-01T19:38:01.280556+00:00 app[web.1]:  "commentableID": 1,
2017-07-01T19:38:01.280556+00:00 app[web.1]:  "hasRelatedCommentable": true,
2017-07-01T19:38:01.280557+00:00 app[web.1]:  "isRepoEvent": true,
2017-07-01T19:38:01.280557+00:00 app[web.1]:  "isTriggeredByUser": true,
2017-07-01T19:38:01.280558+00:00 app[web.1]:  "repo": {
2017-07-01T19:38:01.280559+00:00 app[web.1]:   "fullName": "parse-community/parse-community-peril",
2017-07-01T19:38:01.280559+00:00 app[web.1]:   "id": 111,
2017-07-01T19:38:01.280559+00:00 app[web.1]:   "installationID": 1,
2017-07-01T19:38:01.280560+00:00 app[web.1]:   "rules": {}
2017-07-01T19:38:01.280561+00:00 app[web.1]:  },
2017-07-01T19:38:01.280561+00:00 app[web.1]:  "repoName": "parse-community/parse-community-peril",
2017-07-01T19:38:01.280562+00:00 app[web.1]:  "triggeredByUsername": "flovilmart"
2017-07-01T19:38:01.280562+00:00 app[web.1]: }
2017-07-01T19:38:01.642945+00:00 app[web.1]: info: [runner] - Running Danger: got API? [object Object]
2017-07-01T19:38:02.947898+00:00 app[web.1]: Promise { null } 
2017-07-01T19:38:03.022617+00:00 app[web.1]: info: [runner] - Running Danger: got API? [object Object]
2017-07-01T19:38:04.002010+00:00 heroku[router]: at=info method=POST path="/webhook" host=peril-parse-community.herokuapp.com request_id=3498240c-52a7-4d25-8401-f5a5e3a54d2e fwd="192.30.252.34" dyno=web.1 connect=0ms service=2738ms status=200 bytes=216 protocol=https
2017-07-01T19:38:03.988830+00:00 app[web.1]: Promise { null }
2017-07-01T19:38:03.990379+00:00 app[web.1]: info: [runner] - Commenting, withresults:
2017-07-01T19:38:03.990380+00:00 app[web.1]: mds: 0
2017-07-01T19:38:03.990381+00:00 app[web.1]: mds: 0
2017-07-01T19:38:03.990382+00:00 app[web.1]: warns: 4
2017-07-01T19:38:03.990383+00:00 app[web.1]: fails: 0
2017-07-01T19:38:03.990383+00:00 app[web.1]:
2017-07-01T19:38:04.165922+00:00 app[web.1]: Request failed [403]: https://api.github.com/repos/parse-community/parse-community-peril/statuses/6bfe483dd5b8fd8904b010fb3b86d585d70b460f
2017-07-01T19:38:04.165996+00:00 app[web.1]:   "message": "Resource not accessible by integration",
2017-07-01T19:38:04.165995+00:00 app[web.1]: Response: {
2017-07-01T19:38:04.165997+00:00 app[web.1]:   "documentation_url": "https://developer.github.com/v3"
2017-07-01T19:38:04.165998+00:00 app[web.1]: }
2017-07-01T19:38:04.166164+00:00 app[web.1]: Found only warnings, not failing the build.

@orta
Copy link

orta commented Jul 2, 2017

👍

@steprescott
Copy link

steprescott commented Jul 18, 2017

I am seeing this issue too

2017-07-18T16:04:01.329589+00:00 app[web.1]: info: [runner] - Running Danger: got API? [object Object]
2017-07-18T16:04:04.573321+00:00 app[web.1]: info: [runner] - Running Danger: got API? [object Object]
2017-07-18T16:04:07.410923+00:00 app[web.1]: info: [runner] - Commenting, with results: 
2017-07-18T16:04:07.410939+00:00 app[web.1]: mds: 0
2017-07-18T16:04:07.410940+00:00 app[web.1]: messages: 2
2017-07-18T16:04:07.410941+00:00 app[web.1]: warns: 0
2017-07-18T16:04:07.410942+00:00 app[web.1]: fails: 0

@orta
Copy link

orta commented Aug 12, 2017

I just fixed the duped PR runs

@ghost
Copy link

ghost commented Aug 12, 2018

Warnings
⚠️

Please add a changelog entry for your changes.

Messages
📖

all good!

📖

lets go

📖

{"isOpen":true,"pathForChangelog":{"path":"CHANGELOG.md","mode":"100644","type":"blob","sha":"fe6f18a2750ce850170f13b6eb43566d3b5e9d6f","size":55,"url":"https://api.github.com/repos/parse-community/parse-community-peril/git/blobs/fe6f18a2750ce850170f13b6eb43566d3b5e9d6f"}}

📖

{"isTrivial":false,"isUser":true,"isGreenkeeper":false,"hasCodeChanges":"settings.json"}

Generated by 🚫 dangerJS

@flovilmart flovilmart force-pushed the master branch 18 times, most recently from 6861619 to 59569dc Compare August 18, 2018 03:21
@flovilmart flovilmart closed this May 22, 2019
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.

None yet

3 participants