These are chat archives for meadsteve/white-bread

1st
Feb 2016
Steve B
@meadsteve
Feb 01 2016 18:06
@rkotze also not sure what approach you're taking but my first thought would be to create a macro that calls SuiteConfiguration. add_suite/1 for each feature file it finds
as far as I can tell this should just work as long as you populate the arguments right
all .exs files in features/contexts are autoloaded
Richard Kotze (mobile)
@rkotze
Feb 01 2016 18:28
@meadsteve that was kind of the idea what I was going with but was not sure if you can have all the modules loaded in when the macro runs
but I will look into it
I was thinking of using the suite_run.ex
which would call a 'context_per_feature' module
and that would deal with creating a suite struct
mm but I will see if I can do it all with the macro
aside from that I'm going on a two week holiday from this Wednesday
won't really get time to work on it till im back
Steve B
@meadsteve
Feb 01 2016 18:33
I'm not sure it needs all the modules loaded when the macro runs. You can always deal with missing modules at runtime.
So it should be able to use the existing struct for a suite
Richard Kotze (mobile)
@rkotze
Feb 01 2016 18:34
So if i've not made a pull request by end of tomorrow
Steve B
@meadsteve
Feb 01 2016 18:34
and just pre-populate it's values
Richard Kotze (mobile)
@rkotze
Feb 01 2016 18:34
then im fine if you or someone else wants to take it on
okay cool, sounds simple enough
I'll give it a try
Steve B
@meadsteve
Feb 01 2016 18:35
@rkotze well I'm happy to wait for you if you're keen to do it. It's not a feature I'd use so it's not blocking me
So it's your call really
Richard Kotze (mobile)
@rkotze
Feb 01 2016 18:35
cool, I'm keen to work on it
just short on time :D
Steve B
@meadsteve
Feb 01 2016 18:36
cool! I know the feeling.
Richard Kotze (mobile)
@rkotze
Feb 01 2016 18:36
yeah its not blocking me at the mo
Steve B
@meadsteve
Feb 01 2016 18:36
and it's always better to have someone who wants/uses a feature to implement it. You'll have a much better idea of whether it fits the need or not
so I view this as a positive
Richard Kotze (mobile)
@rkotze
Feb 01 2016 18:37
nice one :+1:
Richard Kotze (mobile)
@rkotze
Feb 01 2016 21:44
@meadsteve are we just updating the readme for documentation for now?