Notification time stamped 2018-10-01 16:09:36 UTC
==================================================================================
#ansible-meeting: AnsibleFest Austin: Contributors Summit - Contributor Experience
==================================================================================
Meeting started by gundalow at 15:11:43 UTC. The full logs are available
at
https://meetbot.fedoraproject.org/ansible-meeting/2018-10-01/ansiblefest_...
.
Meeting summary
---------------
* Agenda & docs
https://etherpad.openstack.org/p/ansible-community-experience
(gundalow, 15:12:15)
* Docs for how to add integration tests need work (gundalow, 15:19:27)
*
https://raw.githubusercontent.com/ansible/community/master/meetings/ical/...
<- Testing Working Group meeting agenda (gundalow, 15:20:05)
* ACTION: nitzmahone give gundalow docs on how to pip install from a
branch (which also rebases) (gundalow, 15:25:09)
* ACTION: gundalow to review and write docs from
https://github.com/jctanner/ansible-tools/blob/master/ansible-pr-checkout
(gundalow, 15:26:09)
* document a testing workflow based around:
https://help.github.com/articles/checking-out-pull-requests-locally/
(abadger1999, 15:28:22)
* 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)
* ACTION: gundalow document runme.sh (willthames, 15:32:37)
* ACTION: gundalow document the magic metadata stuff (willthames,
15:33:04)
* ACTION: gundalow Document CI test aliases (sdoran, 15:34:34)
* 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)
* ACTION: gundalow runme.sh docs - If you are reading this you probs
don't want this (gundalow, 15:39:20)
* ACTION: gundalow document code coverage (gundalow, 15:39:30)
* LINK:
http://ansibullbot.eng.ansible.com/ansibot/metadata/byfile_sorted.html
(jtanner, 15:44:27)
* Question: Should people with Commit powers update PR for someone else
(gundalow, 15:48:34)
* Question: If a PR only needs a simple fix before merge should that be
fixed then merged (gundalow, 15:51:58)
* Add more into Sanity tests for these things (gundalow, 15:52:13)
* If it's a PR older than n-months then asking them the author to update
isn't good (gundalow, 15:54:04)
* Can we automate `version_added` (gundalow, 15:54:28)
* 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)
* Should we add commits to PRs that we will not merge - Things are lost
by force push? (gundalow, 15:59:36)
* 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)
* LINK:
https://github.com/ansible/ansibullbot/blob/master/ansibullbot/triagers/p...
(jtanner, 16:03:35)
* LINK:
https://developer.github.com/v4/previews/#hovercards (jtanner,
16:06:12)
Meeting ended at 16:09:36 UTC.
Action Items
------------
* nitzmahone give gundalow docs on how to pip install from a branch
(which also rebases)
* gundalow to review and write docs from
https://github.com/jctanner/ansible-tools/blob/master/ansible-pr-checkout
* gundalow document runme.sh
* gundalow document the magic metadata stuff
* gundalow Document CI test aliases
* gundalow to talk mattclay to de duplicate Ansibullbot comments -
especially for same test failing on different Python versions -PR
46344
* gundalow runme.sh docs - If you are reading this you probs don't
want this
* gundalow document code coverage
Action Items, by person
-----------------------
* gundalow
* nitzmahone give gundalow docs on how to pip install from a branch
(which also rebases)
* gundalow to review and write docs from
https://github.com/jctanner/ansible-tools/blob/master/ansible-pr-checkout
* gundalow document runme.sh
* gundalow document the magic metadata stuff
* gundalow Document CI test aliases
* gundalow to talk mattclay to de duplicate Ansibullbot comments -
especially for same test failing on different Python versions -PR
46344
* gundalow runme.sh docs - If you are reading this you probs don't
want this
* gundalow document code coverage
* mattclay
* gundalow to talk mattclay to de duplicate Ansibullbot comments -
especially for same test failing on different Python versions -PR
46344
* nitzmahone
* nitzmahone give gundalow docs on how to pip install from a branch
(which also rebases)
* **UNASSIGNED**
* (none)
People Present (lines said)
---------------------------
* jlk (44)
* gundalow (27)
* jtanner (24)
* sdoran (23)
* geerlingguy (17)
* abadger1999 (15)
* bcoca (13)
* dag (10)
* defionscode (9)
* webknjaz (8)
* zodbot (6)
* ericsysmin (5)
* maxamillion (5)
* dmsimard (4)
* willthames (4)
* tima (4)
* Pilou (3)
* sivel (3)
* felixfontein (2)
* resmo (1)
* misc (1)
* corvus (1)
* shertel (1)
* mordred (1)
* Qalthos (0)
* mattclay (0)
* trishnag (0)
* alikins (0)
* shepdelacreme (0)
* nitzmahone (0)
* thaumos (0)
Generated by `MeetBot`_ 0.1.4
.. _`MeetBot`:
http://wiki.debian.org/MeetBot
https://meetbot.fedoraproject.org/ansible-meeting/2018-10-01/ansiblefest_...
--
You received this message due to your preference settings at
https://apps.fedoraproject.org/notifications/fmnmeetingminutes.id.fedorap...