Please note that #1 is now merged into master. This work by Maxime removes Highcharts from Airflow bringing us closes to an Apache release. Some charts are not supported anymore (Max any details?), but I assume we can add those in again if needed.
- Bolke > Op 14 jun. 2016, om 20:15 heeft Bolke de Bruin <[email protected]> het > volgende geschreven: > > Nice! Why don’t we also merge 1 into master then and fix issues from there? > It will get wider exposure (We run master in our dev environments for > example), it won’t put everything into the lab or Airbnb. Master hardly > should be considered production so imho it is allowed some time to settle > before it is stable again. > > Does someone want to pick up 2 say by the end of week or next week to fix all > headers? > > - Bolke > > >> Op 14 jun. 2016, om 20:01 heeft Maxime Beauchemin >> <[email protected]> het volgende geschreven: >> >> I think (1) is good to go. I can cherry pick it into our production to make >> sure it's ready for release. >> >> (2) should be fine, git should be able to merge easily, otherwise it's >> super easy to resolve any conflicts >> >> Max >> >> On Tue, Jun 14, 2016 at 9:04 AM, Chris Riccomini <[email protected]> >> wrote: >> >>> Hey Bolke, >>> >>> I think your list is good. (1) is what I'm most concerned about, as it >>> requires actually touching the code, and is blocking on graduation. I >>> *think* Max had a partial PR on that, but don't know the current state. >>> >>> Re: (2), agree. Should just do a bulk PR for it. >>> >>> Cheers, >>> Chris >>> >>> On Tue, Jun 14, 2016 at 8:41 AM, Bolke de Bruin <[email protected]> wrote: >>> >>>> Hi, >>>> >>>> I am wondering what needs to be done to get to an Apache release? I think >>>> now 1.7.1.3 is out we should be focused on getting one out as we are kind >>>> of half way the incubation process. What comes to my mind is: >>>> >>>> 1. Replace highcharts by D3 (WIP: >>>> https://github.com/apache/incubator-airflow/pull/1469) >>>> 2. Add license headers everywhere (TM) (Sucks, as it will break many PRs >>> - >>>> but lets do it quickly) >>>> 3. Have a review by Apache >>>> >>>> Anything I am missing? >>>> >>>> - Bolke >>> >
