pagure

#1047 Private repo implimentation
Closed by pingou. Opened by farhaan.
farhaan/pagure private-repo  into  master

Download 1047.patch
no initial comment

Too many empty lines here (check pep8)

Let's keep the style consistent here and provide one argument per line

Length is right?

It is better to add the new arguments at the end in case people are using positional arguments instead of using the keyword.

This will require an alembic migration script

1 new commit added

  • Add alembic migration and clean code

You can't set it as nullable=False directly since there are already data in the table

1 new commit added

  • Fix alembic script

Ok, let's rebase and we can merge :)

rebased

Ok, I created a private_repo branch, I'm going to close this PR, could you re-open it against the private_repo branch?

Pull-Request has been closed 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.