Skip to content

Add forbidden error - #30

Merged
jujubot merged 1 commit into
juju:masterfrom
geoffreybauduin:forbidden
Jun 20, 2017
Merged

Add forbidden error#30
jujubot merged 1 commit into
juju:masterfrom
geoffreybauduin:forbidden

Conversation

@geoffreybauduin

Copy link
Copy Markdown
Contributor

The "forbidden" error is missing, and since most of the common http errors are implemented, I believe it'd be nice to add this one.

@axw axw left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@axw

axw commented Jun 20, 2017

Copy link
Copy Markdown
Contributor

$$merge$$

@jujubot

jujubot commented Jun 20, 2017

Copy link
Copy Markdown
Contributor

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju-errors

@jujubot

jujubot commented Jun 20, 2017

Copy link
Copy Markdown
Contributor

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju-errors/10

@axw

axw commented Jun 20, 2017

Copy link
Copy Markdown
Contributor

$$merge$$

@jujubot

jujubot commented Jun 20, 2017

Copy link
Copy Markdown
Contributor

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju-errors

@jujubot

jujubot commented Jun 20, 2017

Copy link
Copy Markdown
Contributor

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju-errors/11

@jujubot
jujubot merged commit a3edbf4 into juju:master Jun 20, 2017
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.

3 participants