Re: [Development] HEADS-UP: Qt 6.7 API change review

[Jani Heikkinen via Development]

Hi all,

API review for Qt 6.7 is still ongoing, see 
https://bugreports.qt.io/browse/QTBUG-119952

Especially qtbase has many documents open, but there are also open ones in 
qtdeclarative, qtquick3d, qtsensors and qtwebengine.

Please help us to complete the review; we need to do it immediately to be able 
to keep the Qt 6.7.0 release schedule

br,
Jani

From: Development  On Behalf Of Jani 
Heikkinen via Development
Sent: perjantai 5. tammikuuta 2024 7.50
To: development@qt-project.org
Subject: [Development] HEADS-UP: Qt 6.7 API change review


Hi!



Qt 6.7 API Change Review is finally started, see 
https://bugreports.qt.io/browse/QTBUG-119952

generated diffs are here:

  *   https://codereview.qt-project.org/q/topic:api-change-review-6.7
  *   https://codereview.qt-project.org/q/topic:qml_api_review_6.7



Please review the diffs as soon as possible.



br,

Jani

-- 
Development mailing list
Development@qt-project.org
https://lists.qt-project.org/listinfo/development

Re: [Development] HEADS-UP: Qt 6.7 API change review

[Marc Mutz via Development]

A lot has happened over at 
https://wiki.qt.io/Things_To_Look_Out_For_In_Reviews (thanks to Kai for 
significant additions early on and Eddy and Ahmad for various fixes).

It would be nice if this could be less of a one-man show, though, so, 
please, peruse and contribute!

Thanks,
Marc

On 05.01.24 09:28, Marc Mutz via Development wrote:
> Hi,
> 
> In the hope of spreading knowledge, I've created a staging area in the
> wiki called "Things to look out for in reviews." This is a
> low-entry-barrier way to quickly publish a guideline. After each
> release, we should discuss where to move these "things" to, see the page
> for details:
> 
> https://wiki.qt.io/Things_To_Look_Out_For_In_Reviews
> 
> Feel free to contribute, either with your own guidelines, or with
> contributions to the Talk page.
> 
> I'll keep on adding as I find stuff.
> 
> I had a similar page back at KDAB, which ended up seeding the initial
> Clazy rule set.
> 
> Thanks,
> Marc
> 
> On 05.01.24 06:49, Jani Heikkinen via Development wrote:
>> Hi!
>>
>> Qt 6.7 API Change Review is finally started, see
>> https://bugreports.qt.io/browse/QTBUG-119952
>> 
>>
>> generated diffs are here:
>>
>>* https://codereview.qt-project.org/q/topic:api-change-review-6.7
>>  
>>* https://codereview.qt-project.org/q/topic:qml_api_review_6.7
>>  
>>
>> Please review the diffs as soon as possible.
>>
>> br,
>>
>> Jani
>>
>>
-- 
Marc Mutz 
Principal Software Engineer

The Qt Company
Erich-Thilo-Str. 10 12489
Berlin, Germany
www.qt.io

Geschäftsführer: Mika Pälsi, Juha Varelius, Jouni Lintunen
Sitz der Gesellschaft: Berlin,
Registergericht: Amtsgericht Charlottenburg,
HRB 144331 B

-- 
Development mailing list
Development@qt-project.org
https://lists.qt-project.org/listinfo/development

Re: [Development] HEADS-UP: Qt 6.7 API change review

[Marc Mutz via Development]

Hi,

In the hope of spreading knowledge, I've created a staging area in the 
wiki called "Things to look out for in reviews." This is a 
low-entry-barrier way to quickly publish a guideline. After each 
release, we should discuss where to move these "things" to, see the page 
for details:

https://wiki.qt.io/Things_To_Look_Out_For_In_Reviews

Feel free to contribute, either with your own guidelines, or with 
contributions to the Talk page.

I'll keep on adding as I find stuff.

I had a similar page back at KDAB, which ended up seeding the initial 
Clazy rule set.

Thanks,
Marc

On 05.01.24 06:49, Jani Heikkinen via Development wrote:
> Hi!
> 
> Qt 6.7 API Change Review is finally started, see 
> https://bugreports.qt.io/browse/QTBUG-119952 
> 
> 
> generated diffs are here:
> 
>   * https://codereview.qt-project.org/q/topic:api-change-review-6.7
> 
>   * https://codereview.qt-project.org/q/topic:qml_api_review_6.7
> 
> 
> Please review the diffs as soon as possible.
> 
> br,
> 
> Jani
> 
> 
-- 
Marc Mutz 
Principal Software Engineer

The Qt Company
Erich-Thilo-Str. 10 12489
Berlin, Germany
www.qt.io

Geschäftsführer: Mika Pälsi, Juha Varelius, Jouni Lintunen
Sitz der Gesellschaft: Berlin,
Registergericht: Amtsgericht Charlottenburg,
HRB 144331 B

-- 
Development mailing list
Development@qt-project.org
https://lists.qt-project.org/listinfo/development

Re: [Development] HEADS-UP: Qt 6.7 API change review

[Kai Köhne via Development]

Hi,

also a reminder from my side: When checking out any API changes, please also 
check the API _documentation_. 

A good start for this is https://doc-snapshots.qt.io/qt6-6.7/newclasses67.html

New API
* Is it documented, and marked with \since?
* Is the documentation in good shape (grammar, style, completeness)? 
* Is it interlinked with related API, and related overview documentation?
* Should it be mentioned on https://doc-snapshots.qt.io/qt6-6.7/whatsnew67.html?

Deprecated API
* Is the API also deprecated in the documentation (including the Qt version it 
is first deprecated)?
* Is it explained why it is deprecated, or what to use instead?
* Should it be mentioned on https://doc-snapshots.qt.io/qt6-6.7/whatsnew67.html?

If in doubt, please file a bug on JIRA or reach out to one of the documentation 
writers.

Thanks!

Kai


From: Development  on behalf of Jani 
Heikkinen via Development 
Sent: Friday, January 5, 2024 6:49
To: development@qt-project.org
Subject: [Development] HEADS-UP: Qt 6.7 API change review

Hi!



Qt 6.7 API Change Review is finally started, see 
https://bugreports.qt.io/browse/QTBUG-119952

generated diffs are here:

  *   https://codereview.qt-project.org/q/topic:api-change-review-6.7
  *   https://codereview.qt-project.org/q/topic:qml_api_review_6.7



Please review the diffs as soon as possible.



br,

Jani

-- 
Development mailing list
Development@qt-project.org
https://lists.qt-project.org/listinfo/development

[Development] HEADS-UP: Qt 6.7 API change review

[Jani Heikkinen via Development]

Hi!



Qt 6.7 API Change Review is finally started, see 
https://bugreports.qt.io/browse/QTBUG-119952

generated diffs are here:

  *   https://codereview.qt-project.org/q/topic:api-change-review-6.7
  *   https://codereview.qt-project.org/q/topic:qml_api_review_6.7



Please review the diffs as soon as possible.



br,

Jani

-- 
Development mailing list
Development@qt-project.org
https://lists.qt-project.org/listinfo/development