search

NiklasRosenstein / flux-ci

Flux is your own private & lightweight CI server.
MIT License
26 stars 10 forks source link

Add option to load flux config specified by cmdline arg #59

Closed gsantner closed 6 years ago

gsantner commented 6 years ago

Add option to load flux config specified by cmdline arg

gsantner commented 6 years ago

Honestly - I think this whole thing is anyway botched. This should be a normal config file, not a python executeable/script, and not loaded into global namespace. My opinion on this

gsantner commented 6 years ago

Did you hit the wrong button? :D

NiklasRosenstein commented 6 years ago

Using just Python code for configuration gives a lot of flexibility.

NiklasRosenstein commented 6 years ago

Did you hit the wrong button? :D

I made some changes on top of the PR ;)

https://github.com/NiklasRosenstein/flux-ci/commit/75a7e3a19620927b1bb19319e2fce23be2dc4171

gsantner commented 6 years ago

Ah okay! :+1: :D