Oh here's something. Somewhere inside the source release I have a version string that says "0.34.0rc1" and that now I'm realizing should really say "0.34.0" as it becomes an official release. Now changing that string will make for a different SHA.
How should I go about this? On Wed, Aug 21, 2019 at 9:54 PM Maxime Beauchemin < maximebeauche...@gmail.com> wrote: > Thanks Justin, I just addressed 1,2 and 3 here > https://github.com/apache/incubator-superset/pull/8087, but since these > issues seems non-blocking (given your +1) the fixes won't make 0.34.0 which > looks like it is good to go. > > I couldn't get a clear read on the monokai theme's license either, I'll > see with Erik who introduced these line whether we can just remove the > lines. > > Oh, mentors, quick question about the [RESULT] post, do I account for both > threads (dev@+general@) in it, or just the general@ one? > > Max > > On Tue, Aug 20, 2019 at 8:05 PM Justin Mclean <jus...@classsoftware.com> > wrote: > >> Hi, >> >> +1 (binding) >> >> I checked: >> - incubating in name >> - signatures and hashes fine >> - DISCLAIMER exists, although you may want to use the DISCLAIMER_WIP [8] >> - LICENSE is probably missing a few things (se below) >> - NOTICE is fine >> - No unexpected binary files >> - I didn’t check compiling from source >> >> LiCENSE is missing: >> - This BSD licensed file [1] The file also has an incorrectly has an ASF >> header on it. >> - This file in not mentioned [2] and I think is licensed like so [3] If >> so that may be problematic [4]. However I seem to recall this was discussed >> before so there may of been some conclusion about it? I also note that >> .*geojson are explicitly listed in your rat excludes so I assume there a >> reason for that. (Comments in the rat exclusion file could help here). >> - This file [5] looks to contain the Monokai theme from [6] I’m unsure >> how this is licensed. The pro version seems to require a commercial >> license. It might of come from here [7]? which is MIT licensed. >> >> Thanks, >> Justin >> >> 1. ./superset/extract_table_names.py >> 2 ./assets/src/visualizations/CountryMap/countries/india.geojson >> 3. ./licenses/LICENSE-diva-gis.txt >> 4. https://apache.org/legal/resolved.html#cc-by >> 5. superset/assets/src/components/FilterableTable/FilterableTable.jsx >> 6. https://www.monokai.nl >> 7 https://packagecontrol.io/packages/Monokai%20JSON%2B >> 8 https://incubator.apache.org/policy/incubation.html#disclaimers >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: general-unsubscr...@incubator.apache.org >> For additional commands, e-mail: general-h...@incubator.apache.org >> >>