#ansible-meeting: AnsibleFest Austin: Contributors Summit - Contributor Experience

Meeting started by gundalow at 15:11:43 UTC (full logs).

Meeting summary

    1. Agenda & docs https://etherpad.openstack.org/p/ansible-community-experience (gundalow, 15:12:15)
    2. Docs for how to add integration tests need work (gundalow, 15:19:27)
    3. https://raw.githubusercontent.com/ansible/community/master/meetings/ical/testing.ics <- Testing Working Group meeting agenda (gundalow, 15:20:05)
    4. ACTION: nitzmahone give gundalow docs on how to pip install from a branch (which also rebases) (gundalow, 15:25:09)
    5. ACTION: gundalow to review and write docs from https://github.com/jctanner/ansible-tools/blob/master/ansible-pr-checkout (gundalow, 15:26:09)
    6. document a testing workflow based around: https://help.github.com/articles/checking-out-pull-requests-locally/ (abadger1999, 15:28:22)
    7. Bot could add a single comment on PRs saying "Docs for PR author (thanks, how to add tests) PR authors (How to download and test, how to report issues) (gundalow, 15:30:41)
    8. ACTION: gundalow document runme.sh (willthames, 15:32:37)
    9. ACTION: gundalow document the magic metadata stuff (willthames, 15:33:04)
    10. ACTION: gundalow Document CI test aliases (sdoran, 15:34:34)
    11. ACTION: gundalow to talk mattclay to de duplicate Ansibullbot comments - especially for same test failing on different Python versions -PR 46344 (gundalow, 15:36:37)
    12. ACTION: gundalow runme.sh docs - If you are reading this you probs don't want this (gundalow, 15:39:20)
    13. ACTION: gundalow document code coverage (gundalow, 15:39:30)
    14. http://ansibullbot.eng.ansible.com/ansibot/metadata/byfile_sorted.html (jtanner, 15:44:27)
    15. Question: Should people with Commit powers update PR for someone else (gundalow, 15:48:34)
    16. Question: If a PR only needs a simple fix before merge should that be fixed then merged (gundalow, 15:51:58)
    17. Add more into Sanity tests for these things (gundalow, 15:52:13)
    18. If it's a PR older than n-months then asking them the author to update isn't good (gundalow, 15:54:04)
    19. Can we automate `version_added` (gundalow, 15:54:28)
    20. jlk: I do think Ansible should take the approach other projects have taken. "Your submission is great and it is nearly complete. Thank you, we'll fix up the rest and merge it." Focus on getting the code in rather than doing back/forth to have the contributor have a "perfect" contributio (gundalow, 15:56:36)
    21. Should we add commits to PRs that we will not merge - Things are lost by force push? (gundalow, 15:59:36)
    22. First PR comment for new_contributors, give them some extra guidance, extra thanks. Make it clearer to reviews that this is a new contributor and may need guidance, https://developer.github.com/changes/2018-03-21-hovercard-api-preview/ (gundalow, 16:03:25)
    23. https://github.com/ansible/ansibullbot/blob/master/ansibullbot/triagers/plugins/contributors.py (jtanner, 16:03:35)
    24. https://developer.github.com/v4/previews/#hovercards (jtanner, 16:06:12)


Meeting ended at 16:09:36 UTC (full logs).

Action items

  1. nitzmahone give gundalow docs on how to pip install from a branch (which also rebases)
  2. gundalow to review and write docs from https://github.com/jctanner/ansible-tools/blob/master/ansible-pr-checkout
  3. gundalow document runme.sh
  4. gundalow document the magic metadata stuff
  5. gundalow Document CI test aliases
  6. gundalow to talk mattclay to de duplicate Ansibullbot comments - especially for same test failing on different Python versions -PR 46344
  7. gundalow runme.sh docs - If you are reading this you probs don't want this
  8. gundalow document code coverage


Action items, by person

  1. gundalow
    1. nitzmahone give gundalow docs on how to pip install from a branch (which also rebases)
    2. gundalow to review and write docs from https://github.com/jctanner/ansible-tools/blob/master/ansible-pr-checkout
    3. gundalow document runme.sh
    4. gundalow document the magic metadata stuff
    5. gundalow Document CI test aliases
    6. gundalow to talk mattclay to de duplicate Ansibullbot comments - especially for same test failing on different Python versions -PR 46344
    7. gundalow runme.sh docs - If you are reading this you probs don't want this
    8. gundalow document code coverage
  2. mattclay
    1. gundalow to talk mattclay to de duplicate Ansibullbot comments - especially for same test failing on different Python versions -PR 46344
  3. nitzmahone
    1. nitzmahone give gundalow docs on how to pip install from a branch (which also rebases)


People present (lines said)

  1. jlk (44)
  2. gundalow (27)
  3. jtanner (24)
  4. sdoran (23)
  5. geerlingguy (17)
  6. abadger1999 (15)
  7. bcoca (13)
  8. dag (10)
  9. defionscode (9)
  10. webknjaz (8)
  11. zodbot (6)
  12. ericsysmin (5)
  13. maxamillion (5)
  14. dmsimard (4)
  15. willthames (4)
  16. tima (4)
  17. Pilou (3)
  18. sivel (3)
  19. felixfontein (2)
  20. resmo (1)
  21. misc (1)
  22. corvus (1)
  23. shertel (1)
  24. mordred (1)
  25. Qalthos (0)
  26. mattclay (0)
  27. trishnag (0)
  28. alikins (0)
  29. shepdelacreme (0)
  30. nitzmahone (0)
  31. thaumos (0)


Generated by MeetBot 0.1.4.