Outcomes from PR Triage on November 12, 2013

[Andy Parker]

At 9am Pacific we had a public hangout to go over the puppet/facter/hiera/stdlib open pull requests. Present were: me, Josh Cooper, Kylo Ginsberg, Henrik Lindberg, Josh Hoblitt, and Ashley Penney.

1962: Needs more investigation to understand why the default compiler's method isn't working. (Adrien later said that there seems to be some quirk in the indirector code).
1957: Closed since we aren't going forward with this implementation of data in modules.
1944: Tough call. It changes semantics and doesn't honor the system parameter. Closed with a suggestion of trying the system parameter approach.
1942: Have a concern that it checking for the shell existing could cause problems. We need to look at it in more depth to bring it in. There are 5 different platforms affected. It also looks to be a backwards incompatible change since shell cannot be managed anymore by providers that aren't updated. Kicked back with a recommendation to simplify to just affect the useradd provider.
1930: Closed since we are not going down that path.
1916: Need to merge. Done and added to iteration to track.
1854: Rejected the change, but we recognize the problem and have plans for improvements in that area.
1848: Closed and suggested it be published as a module on the forge.
628: basically, great idea, contributor has moved on, so implement a simple solution to the range dilemma ourselves 

The next scheduled one is Wednesday, November 27th. The link for the hangout will be posted to the #puppet-dev channel shortly before we start. Hope to see some of you there.

--

Andrew Parker

an...@puppetlabs.com

Freenode: zaphod42

Twitter: @aparker42

Software Developer

Join us at PuppetConf 2014, September 23-24 in San Francisco