Amend if files are null then return Null rather than exception

ashwanthkumar / gocd-janitor

Attempt at a more intelligent GoCD purge policy

16 stars 11 forks source link

Amend if files are null then return Null rather than exception #2

Closed wingZero21 closed 8 years ago

wingZero21 commented 8 years ago

Amendment to stop error if there are no pipeline directories present.

ashwanthkumar commented 8 years ago

@wingZero21 This will cause NullPointerException to be thrown when the result is being consumed by the next for-loop. We should return an empty array instead of a null.

ashwanthkumar commented 8 years ago

Since the change is trivial, I pushed it myself via https://github.com/ashwanthkumar/gocd-janitor/commit/91c15a257c8b2deaa12034fd44e1df6175e5f19d. Thanks for the attempt at contributing. Keep them coming :smiley:

Closing this now.