pagure-dist-git

#37 Customize the user's page for pagure over dist-git
Merged by pingou. Opened by pingou.
user_page  into  master

Download 37.patch
no initial comment

space after forks_length,

Shouldn't it be group's name rather than SQLAlchemy object? (in url_for)

same here

probably same thing which we did with fullname? i.e. show fullname and username in ().

awesome!

space before +

space before {

space before {

one more indentation level

indent++

space before {

space before {

why not to use <script ... /> instead of <script ...> </script>?

These are already the group names :)

Legacy habits, there used to be a difference (not sure if that's still the case)

No I indent on 2 spaces in the html/js parts :)

1 new commit added

  • Code style fixes

I think I addressed all the ones I agree with :)

: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.