In D58234#1336907, @salvadore wrote:

I approved the whole report, but there there is something weird with this review, it looks like it is referencing a commit that fixed line 25 instead of a commit that adds the full file.

I approved the whole report, but there there is something weird with this review, it looks like it is referencing a commit that fixed line 25 instead of a commit that adds the full file.

Looks good.

remark considered

Great! Thanks, Vladlen!

Approved, thanks!

LGTM. Thanks for taking care of this!

Approved, thanks!

previous comments applied to the text

Well done. Thanks a lot, vladlen!

all proposed changes applied

LGTM.

three small corrections, otherwise looks good, thank you

Thanks! Approved once the mailto macro is used

Looks good.

considered all comments from the team

All comments from team members are considered. It final version

It looks good.

Changes are done according to comments.

Thank you for your submission. A few updates suggested for syntax fixes.

Approved, thanks!

Fix: pane(s) -> panel(s)

Yes, this information is in other report entries, so this can be closed.

I am a bit confused about this review (that I had totally forgotten, sorry).
Does it actually need to be merged? These reports are very short and do not add much to the other reports we already have. In particular we already have a clusteradm report and, hopefully, @lwhsu is about to send a ci report.

• Split into different files
• As I am also here mention about multiple staffs in CI and also all of the CI works are mentioned in the CI status report. So refer to them.
• Added a ports file where I have mentioned the work I have done for the ports.
• Added a cluster file where I have mentioned about deploying the Grimoirelab.
• Totally skipped mentioning about CIS as it does not fall in any groups.

In D48392#1103774, @jrm wrote:

In D48392#1103769, @bofh wrote:

In D48392#1103768, @jrm wrote:

This was submitted based on a miscommunication. I'm preparing the Foundation entry, but waiting on some authors to share their contributions. @bofh, could you please close this revision?

I am working on an updated patch as suggested by you. Or do you want me to close the revision and send you the text?

I requested that you submit one or more report entries that are specific to your work like the other Foundation developers. For example:

• https://cgit.freebsd.org/doc/tree/website/content/en/status/report-2024-10-2024-12/suspend.adoc
• https://cgit.freebsd.org/doc/tree/website/content/en/status/report-2024-10-2024-12/wireless-iwx.adoc
• https://cgit.freebsd.org/doc/tree/website/content/en/status/report-2024-10-2024-12/audio.adoc
• https://cgit.freebsd.org/doc/tree/website/content/en/status/report-2024-10-2024-12/foundation-security.adoc

...

If you plan to completely overwrite this review with your content, that also works.

In D48392#1103769, @bofh wrote:

In D48392#1103768, @jrm wrote:

This was submitted based on a miscommunication. I'm preparing the Foundation entry, but waiting on some authors to share their contributions. @bofh, could you please close this revision?

I am working on an updated patch as suggested by you. Or do you want me to close the revision and send you the text?

In D48392#1103768, @jrm wrote:

This was submitted based on a miscommunication. I'm preparing the Foundation entry, but waiting on some authors to share their contributions. @bofh, could you please close this revision?