pagure

#1267 Multiple fix to get pagure_ci up and running
Merged by pingou. Opened by pingou.
fix_pagure_ci  into  master

Download 1267.patch
no initial comment

1 new commit added

  • Make the REPO variable be the public url where it can be cloned

1 new commit added

  • Fix retrieving the username from the data when the repo is a fork
Edited by vivekanand1101

why don't i find any reference of this in model? ( i mean ci_hook)

... because it's not the place i should be looking :'(
Edited by vivekanand1101

this one is extra

Indeed

1 new commit added

  • Drop un-used variable

rebased

@vivekanand1101 did you see the latest version?

shouldn't this method be used to get the project ?

shouldn't this method be used to get the project ?

That method can actually be removed, it's not longer used anywhere.

It was decided to rely on username/name to find the project and then check if the provided CI token matches rather than just relying on the token.

rebased

1 new commit added

  • Drop get_project_by_ci_token from pagure.lib_ci since it not used anywhere

:thumbsup:

Thanks :)

Pull-Request has been merged by pingou
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.