pagure

#1069 Adjust is_watching and fix the tests related to this feature
Merged by pingou. Opened by pingou.
fix_test_watch  into  master

Download 1069.patch
no initial comment

1 new commit added

  • Remove a chunk of code that's not needed

isn't user == user_obj?

Add a space here

Same, space missing

Typo here watch

space missing in all the assignments for watch

Looks good to me :thumbsup:

Actually we can return True from here itself and False in the end then we don't need to have the variable watch

no here user is flask.g.fas_user

rebased

Good catch, I will adjust :)

2 new commits added

  • Fix typo watch2->watch and pep8 with assigning the watch variable
  • Simplify the end of the is_watching method as @sayanchowdhury suggested

:thumbsup: for me

Thanks

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.