Qt wiki will be updated on October 12th 2023 starting at 11:30 AM (EEST) and the maintenance will last around 2-3 hours. During the maintenance the site will be unavailable.
Code Review: Sharing the load: Difference between revisions
Jump to navigation
Jump to search
(Nils Jeisecke's notes, via e-mail) |
m (Initial wikification) |
||
Line 1: | Line 1: | ||
[[Category:QtCS2019]] | [[Category:QtCS2019]] | ||
* Situation: Many reviews in queue for long time (forever?) | * Situation: Many reviews in queue for long time (forever?) | ||
** Android: They have a button: Find "Maintainers" (Plugin for Gerrit) | |||
** Thiago: Would solve the first 15 Minutes for Contributors, not enough | |||
** Improve the culture, appreciate the effort ("Thank you!") | |||
** Get more people on-board as reviewers (not as approvers yet) | |||
** Idea: Should not be +1 and +1, should be "Thumbs up" and "Tick!" | |||
*** Some reviewers don't understand the implication of "+1" (two "+1" != "+2") | |||
* What does a good patch looks like? | * What does a good patch looks like? | ||
** breaking patch up into multiple patches | |||
** Is the documentation really good enough (opinions vary) | |||
* Ask contributors whose changes have just been review to review others (sometimes happens) | * Ask contributors whose changes have just been review to review others (sometimes happens) | ||
* Gerrit | * Gerrit | ||
** Sanity bot is good | |||
** Revive early warning bot! | |||
*** Should changes only be approved after tests has run on CI? | |||
*** Can we get a "small CI" run (just Linux, "does it build?") | |||
*** Should the bot automatically run tests if tests have been changed? | |||
*** Should the bot automatically give a warning if tests have not been added (excluding doc only changes)? | |||
** "Auto defer" bot is currently not active, would be useful to automatically defer patches where contributor did not response to requests. | |||
** Automatically stage when change is approved? Faster turnaround time. | |||
** Mail (monthly?) reports to maintainers (or others?) (based on gerrit dashboard config?): "N Changes approved, but not staged? Link to staging page", List per module? | |||
** Let's try it and see how it works out | |||
* Gerrit Features unused right now | * Gerrit Features unused right now | ||
** Attach labels | |||
** Bot could apply hashtags ("wants adoption", "stage-me") | |||
** Has a feature to make clazy automatically appply improvements to the change ("contributor has to click a button") | |||
* Public Community Feedback does not exist | * Public Community Feedback does not exist | ||
** Alex: Use Thiago's script to highlight new contributions (Grafana?)? "Welcome new contributor!" for blog post (Like KDE does)? | |||
--- | --- | ||
Line 40: | Line 39: | ||
* Give contributor feedback | * Give contributor feedback | ||
** Bot should say: "Welcome, Thanks for your first contribution" | |||
** Monthly blog post advertising external contributors? | |||
** Send out physical awards? T-Shirts, "Thank you" letter, …? | |||
* Make contributing easier | * Make contributing easier | ||
** Put contributor documentation into Qt's documentation ("Gettings started")? | |||
** Link freenode IRC Web-Backend to simplify access to IRC for new contributors | |||
** Use Qt account profile to setup everything needed for contributing (ssh keys, CLA check, ...) | |||
** Adapt github workflow (Checklist "Auto test done", "Documentation done", ...) | |||
* Triage bugs | * Triage bugs | ||
** Add Jira label ("for juniors")? | |||
* Community | * Community | ||
** Missing Community manager (Consent: Yes, it's needed) | |||
** Missing "qt-project" entity, kind of a foundation? Funding? | |||
** Redefine: What is the qt-project? | |||
* Activating new contributors | * Activating new contributors | ||
** Communication of Qt (company) targets companies, not community (qt.org ⇒ qt.io ⇒ sales) | |||
** Microsoft pushes its stuff to universities, why not Qt (or qt-project?) | |||
** Qt project needs a proper homepage. With nice, contemporary Howtos, ... |
Revision as of 15:35, 22 November 2019
- Situation: Many reviews in queue for long time (forever?)
- Android: They have a button: Find "Maintainers" (Plugin for Gerrit)
- Thiago: Would solve the first 15 Minutes for Contributors, not enough
- Improve the culture, appreciate the effort ("Thank you!")
- Get more people on-board as reviewers (not as approvers yet)
- Idea: Should not be +1 and +1, should be "Thumbs up" and "Tick!"
- Some reviewers don't understand the implication of "+1" (two "+1" != "+2")
- What does a good patch looks like?
- breaking patch up into multiple patches
- Is the documentation really good enough (opinions vary)
- Ask contributors whose changes have just been review to review others (sometimes happens)
- Gerrit
- Sanity bot is good
- Revive early warning bot!
- Should changes only be approved after tests has run on CI?
- Can we get a "small CI" run (just Linux, "does it build?")
- Should the bot automatically run tests if tests have been changed?
- Should the bot automatically give a warning if tests have not been added (excluding doc only changes)?
- "Auto defer" bot is currently not active, would be useful to automatically defer patches where contributor did not response to requests.
- Automatically stage when change is approved? Faster turnaround time.
- Mail (monthly?) reports to maintainers (or others?) (based on gerrit dashboard config?): "N Changes approved, but not staged? Link to staging page", List per module?
- Let's try it and see how it works out
- Gerrit Features unused right now
- Attach labels
- Bot could apply hashtags ("wants adoption", "stage-me")
- Has a feature to make clazy automatically appply improvements to the change ("contributor has to click a button")
- Public Community Feedback does not exist
- Alex: Use Thiago's script to highlight new contributions (Grafana?)? "Welcome new contributor!" for blog post (Like KDE does)?
---
Follow-up session extensions (lots of overlapping topics)
- Give contributor feedback
- Bot should say: "Welcome, Thanks for your first contribution"
- Monthly blog post advertising external contributors?
- Send out physical awards? T-Shirts, "Thank you" letter, …?
- Make contributing easier
- Put contributor documentation into Qt's documentation ("Gettings started")?
- Link freenode IRC Web-Backend to simplify access to IRC for new contributors
- Use Qt account profile to setup everything needed for contributing (ssh keys, CLA check, ...)
- Adapt github workflow (Checklist "Auto test done", "Documentation done", ...)
- Triage bugs
- Add Jira label ("for juniors")?
- Community
- Missing Community manager (Consent: Yes, it's needed)
- Missing "qt-project" entity, kind of a foundation? Funding?
- Redefine: What is the qt-project?
- Activating new contributors
- Communication of Qt (company) targets companies, not community (qt.org ⇒ qt.io ⇒ sales)
- Microsoft pushes its stuff to universities, why not Qt (or qt-project?)
- Qt project needs a proper homepage. With nice, contemporary Howtos, ...