Comments (3)
Looking at the logs there was definitely and issue enabling hooks:
Mar 5 15:40:52 ip-172-31-137-250 docker/42557c80b096[835]: Sat, 05 Mar 2016 15:40:52 GMT zappr:repo-handler:info 2016-03-05T15:40:52Z jonathanpollack: Reloaded from Github
Mar 5 15:41:32 ip-172-31-137-250 docker/42557c80b096[835]: Sat, 05 Mar 2016 15:41:32 GMT zappr:github:error 2016-03-05T15:41:32Z 404 GET /repos/jonathanpollack/director/hooks { message: 'Not Found',
Mar 5 15:41:32 ip-172-31-137-250 docker/42557c80b096[835]: Sat, 05 Mar 2016 15:41:32 GMT zappr:api:error 2016-03-05T15:41:32Z [Error: Not Found]
...
Mar 5 15:41:32 ip-172-31-137-250 docker/42557c80b096[835]: ::ffff:172.31.15.0 - PUT /api/repos/17866643/approval HTTP/1.1 500 21 - 542.272 ms
...
Mar 5 15:41:35 ip-172-31-137-250 docker/42557c80b096[835]: ::ffff:172.31.15.0 - PUT /api/repos/30529798/approval HTTP/1.1 500 21 - 330.292 ms
from zappr.
Apparently because we're constructing the URL out of $username/$repo
, so Zappr thinks delivero/gitworkflowsamples
is jonathanpollack/gitworkflowsamples
, which does not exist.
Thanks for reporting, we definitely have to fix this.
from zappr.
Right on!
from zappr.
Related Issues (20)
- readthedocs header hides parts of documentation HOT 3
- Making Zappr Github Marketplace ready. HOT 2
- add 503 API errors to trouble shooting section
- Accept webhook url as a config parameter HOT 4
- OneOf Labels Check HOT 5
- Support Label based approver group conditions
- Mandate 'contains-issue-number' in PR's body HOT 1
- Zappr fails on (?:(?i)AND)(?:-|\s)\d+ commit msg pattern HOT 2
- Quiz about Authorization Callback URL HOT 3
- Zappr incorrectly reports a check was enabled by a user
- Approval/veto regexes only work for start of comment HOT 1
- Add prettier to the project.
- Ignore "pull request tasks check" for specified checkboxes
- User-defined regex patterns for specification body options `contains-issue-number` and `contains-url`
- Effective config `specification.title.minimum-length` has property `enablead`
- Incorrectly reported failed check name 'contains-issue-number'
- "Update this Branch" should not require approvals again
- Zappr does not response
- Unexpected rejection of the PR title: "PR's title is too short (8/8)"
- Builds are failing since December, 6th
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from zappr.