#1155 Add docstring for pagure-ci
Merged by pingou. Opened by farhaan.
farhaan/pagure pagure-ci-doc  into  master

Download 1155.patch
no initial comment

@vivekanand1101 review :)

rebased

Exception raised then the hook is inactive

And this should be with the other pagure exceptions and inherit from the base class of pagure exceptions

Btw, could you check the style of the docstrings in pagure.lib and all?

It would be nice to be consistent there

We know it's a function, no need to mention it.
Process build of the project on jenkins... (no idea if this is right or not, adjust as needed)

1 new commit added

  • Move exceptions to pagure exception

Style is still inconsistent

2 new commits added

  • Move exceptions to pagure exception
  • Add docstring for pagure-ci

1 new commit added

  • Fix styling of exception

3 new commits added

  • Fix styling of exception
  • Move exceptions to pagure exception
  • Add docstring for pagure-ci

3 new commits added

  • Fix styling of exception
  • Move exceptions to pagure exception
  • Add docstring for pagure-ci

Looks good to me

@vivekanand1101 are you satisfied as well ? :)

1 new commit added

  • Fix plugin test

rebased

:thumbsup:

rebased

Let's merge ! :)

Pull-Request has been merged by pingou

Metadata