microadam / drone-config-changeset-conditional

The Unlicense
65 stars 17 forks source link

ignore non-yaml configuration files #13

Open bradrydzewski opened 5 years ago

bradrydzewski commented 5 years ago

I recommend returning a 204 no-content and ignore repositories if the configuration file (config_path) does not match .yml or .yaml file extension. This will allow the plugin to play nice with native jsonnet and starlark.

(PS sorry for not being more proactive with a pull request, but the Drone backlog is keeping me too busy right now)

microadam commented 5 years ago

Thanks for the suggestion @bradrydzewski! I will add that in when I have some time