#2553 Allow periods in project names
Closed by pingou. Opened by mprahl.
Unknown source master

Download 2553.patch

This regex caused https://pagure.io/releng/fedora-scm-requests/issue/550 to fail. This PR will allow periods in the project name.

:sparkler:

Thanks @mprahl . Would it be possible to get this into production soon, please?

@pwalter I'm sorry, I don't maintain Pagure so I'm not able to merge PRs or do deployments.

@pingou can you help out with this?

I doubt we want to allow . to be the first character

Hotfixed in prod (just with the second . not the first one).

Could we add tests for this change?

Looks like it's still failing in prod as per https://pagure.io/releng/fedora-scm-requests/issue/550#comment-461892

Closing due to a policy decision on project naming.

And un-hotfixed in prod

Pull-Request has been closed by pingou

Metadata