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

From Qt Wiki
Jump to navigation Jump to search
m (Initial wikification)
m (Minor typo fix and an HTML-ification.)
Line 5: Line 5:
** Improve the culture, appreciate the effort ("Thank you!")
** Improve the culture, appreciate the effort ("Thank you!")
** Get more people on-board as reviewers (not as approvers yet)
** Get more people on-board as reviewers (not as approvers yet)
** Idea: Should not be +1 and +1, should be "Thumbs up" and "Tick!"
** Idea: Should not be +1 and +2, should be "Thumbs up" and "Tick!"
*** Some reviewers don't understand the implication of "+1" (two "+1" != "+2")
*** 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?

Revision as of 15:36, 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 +2, 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, ...