[Development] Adding a .gitreview to all repos
Volker Hilsheimer
volker.hilsheimer at qt.io
Thu Feb 13 12:48:49 CET 2025
> On 12 Feb 2025, at 22:22, Sérgio Martins <iamsergio at gmail.com> wrote:
>
> On Wed, Feb 12, 2025 at 10:44 AM Cristian Le via Development
> <development at qt-project.org> wrote:
>>
>> I want to gather some feedback about adding a .gitreview file for all repos. The file would look like
>>
>> ```
>> [gerrit]
>> host=codereview.qt-project.org
>> project=qt/qtbase
>> defaultbranch=dev
>> ```
>>
>> This would help with out-of-the-box support for some IDEs, e.g. on Jetbrains ones it read this file to determine the default for where to submit the gerrit reviews.
>>
>> If you have git-review cli you also have more simple interface to submit and fetch changes, e.g.
>> ```
>> (Cherry-pick gerrit change)
>> $ git review -x <Change-ID>
>> (Submit a change)
>> $ git review -s 6.9
>> ```
>>
>> For new external contributors this also signals straight away that the projects are managed by gerrit and where to submit the changes.
>>
>> For more reference about the .gitreview check the upstream documentation [1]. There is also a Jira bug I've opened about adding this file [2].
>>
>> [1]: https://docs.opendev.org/opendev/git-review/latest/installation.html#gitreview-file-format
>> [2]: https://bugreports.qt.io/browse/QTBUG-132604
>
> Yes, .gitreview is pretty standard for gerrit and there's no downside
> for whoever isn't using it.
>
>
>
> Regards,
> Sergio Martins
Any change that makes our repos easier to work with using generally available tooling is welcome.
“git-review” is not part of the standard git toolset, but it’s available from at least some package managers. Our (or anyone's, as far as I can tell) git-gpush isn't.
So, adding those files is fine by me; ideally, those additions come with some information on the wiki on how to then set up a workable clone of the Qt repositories.
Volker
More information about the Development
mailing list