Lance Austin
Lance Austin
@WrathOfZombies @Zlatkovsky - I see the change has been committed to master but I'm still seeing this issue. I have 1.02 installed from NPM. Does a new release just need...
Do we want these landing in `release/v2.4` or would it be better to land in `release/v2.5` now?
@LukeShu - as we discussed offline, this fix didn't cover all the cases so I'm going to reopen this and outline what we discussed so we have a record of...
In general, I'm in agreement with this but let's land the rest of the outstanding cycle work first so that we do not end up disrupting cycle work.
@LukeShu @haq204 - I don't see an issue with the proposed changes but I guess why is it that we are keeping the vendor folder around at all? I thought...
@f-herceg - Thanks for this again. As @alexgervais mentioned since this has been a known behavior for awhile now we don't want to treat this as a bugfix because it...
@AliceProxy - per our discussion offline I agree with your call and approach on this.
@kflynn - looks like you have a test failing. Also, it make sense, can we just add a corresponding test to ensure it is working as reported in the issue.
@kflynn - it looks good to me. Let me know what you think of removing the suffix as part of this PR? also, I would like to get a second...
@ddymko - Can you take a look at this, I think it would be good to understand this logic and how we consolidate clusters that have name collisions.