# First code contribution to Ansible, looking for feedback

**URL:** <https://forum.ansible.com/t/first-code-contribution-to-ansible-looking-for-feedback/22677>\
**Category:** Ansible Developer\
**Created:** [September 29, 2016, 3:24pm UTC](https://forum.ansible.com/t/first-code-contribution-to-ansible-looking-for-feedback/22677 "2016-09-29T15:24:48Z")\
**Posts on this page:** 2\
**Page:** 1

<div class="post-metadata">

**Author:** ![Rob\_Phoenix](https://avatars.discourse-cdn.com/v4/letter/r/dec6dc/32.png) [@Rob\_Phoenix](https://forum.ansible.com/u/Rob_Phoenix)\
**Post date:** [September 29, 2016, 3:24pm UTC](https://forum.ansible.com/t/first-code-contribution-to-ansible-looking-for-feedback/22677/1 "2016-09-29T15:24:48Z")

</div>

Hi,

I’m just starting to write some code for a first code PR to Ansible. It’s still a work in progress, but I just want to get some feedback and see if I’m working in the right direction.

It’s in response to this issue:

[https://github.com/ansible/ansible-modules-core/issues/4386](https://github.com/ansible/ansible-modules-core/issues/4386)

My initial exploratory commit is here:

[https://github.com/bordeltabernacle/ansible/commit/34b180cf7f3b5735111d89047ad1f6d7bd169253](https://github.com/bordeltabernacle/ansible/commit/34b180cf7f3b5735111d89047ad1f6d7bd169253)

I’ll look at adding some more checks on the path arg, maybe a try/except. I’m not sure though if there’s anything else to consider, especially outside the method. I assume the functionality in the network modules to use this will be added in another PR, but is there something I’m overlooking or missing?

Any advice/comment, good or bad, will be most welcome.

Thanks,

Rob

---

<div class="post-metadata">

**Author:** ![James\_Cammarata](https://avatars.discourse-cdn.com/v4/letter/j/7c8e57/32.png) [@James\_Cammarata](https://forum.ansible.com/u/James_Cammarata)\
**Post date:** [October 1, 2016, 3:44pm UTC](https://forum.ansible.com/t/first-code-contribution-to-ansible-looking-for-feedback/22677/2 "2016-10-01T15:44:56Z")

</div>

Left one comment on that commit, though Peter or one of the other network team will be the ones merging it.

In general, just go ahead and submit your PR to get it reviewed with feedback, unless it’s about adding a large new feature (in which case having some discussion here on the devel mailing list first may save you some headaches).

Thanks!