Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cutting a release #1657

Closed
JustinBeckwith opened this issue Jul 6, 2018 · 19 comments
Closed

Cutting a release #1657

JustinBeckwith opened this issue Jul 6, 2018 · 19 comments

Comments

@JustinBeckwith
Copy link
Collaborator

Greetings folks! There have been a few critical fixes merged into master recently. I'd very much like to cut an 0.19 release if at all possible.

The biggest issue at the moment is the failing IE9 test suite. As mentioned in #1656, I think it's really worth considering dropping IE9 support. That having been said, if someone can figure out what's up with the tests, I suppose fixing them works too :)

After that comes the release. How are releases orchestrated today? Who's involved in deciding when to ship?

@nickuraltsev @mzabriskie

@traviscrist
Copy link

This would be very helpful for those using typescript as well so that socketPath can be used without building from a hash of github master. The issue was resolved in 524f5bf

@RikkiGibson
Copy link
Contributor

Very strange-- it looks like Edge and Safari support were broken after merging #1689. But I'm sure I had at least the Safari tests passing locally.

Part of the difficulty is I have no idea how to interpret the output I'm seeing on Sauce Labs. Nothing related to the actual Karma output appears to be visible. Anyone more experienced able to comment? @zcei @axe312ger

@emilyemorehouse
Copy link
Member

I've issued the last couple of releases and am happy to continue managing them, I've just been waiting on passing tests. As soon as that's settled, I'll cut the release.

I do have access to run local code using Sauce Labs, so I've been poking at it to see if I can get the tests to behave. My gut was that something changed with the dependencies and is causing it to hang, but I haven't found anything that works.

@axe312ger
Copy link
Contributor

@emilyemorehouse thank you. Everyone able to fix Sauce Labs tests is very welcome. I can try to do it on my own as soon I find time for it :)

@RikkiGibson
Copy link
Contributor

Thanks for the information and please let me know if there’s any way I can assist :)

@emilyemorehouse
Copy link
Member

I've released 0.19.0-beta.1.

This can be installed using npm install axios@0.19.0-beta.1 or npm install axios@next

I'll cut the official 0.19 release when tests are passing 🙂

@RikkiGibson
Copy link
Contributor

Thank you so much!!

@axe312ger
Copy link
Contributor

Looks like IE 11 is green now. Maybe just a Sauce Labs hickup with Safari?

@emilyemorehouse
Copy link
Member

We have our green checkmark back!
screen shot 2018-08-10 at 9 23 25 pm

I'm going to give the beta a couple of weeks before releasing 0.19 -- there's already #1718 which points out a possible regression.

@edgesite
Copy link

edgesite commented Aug 11, 2018

I'd like to roll out a fix to solve #1718 next week, still need more test to release 0.19.

@axe312ger
Copy link
Contributor

So we got 2 customers confirming that the Axios works well in their projects, our internal tests also worked out pretty well.

I guess we can release v0.19 as non-beta? @emilyemorehouse

Here are the customer replies:
contentful/contentful.js#273 (comment)
contentful/contentful.js#272 (comment)

@axe312ger
Copy link
Contributor

(The bundle size was not affected in our case, but we updated other dependencies as well: https://bundlephobia.com/result?p=contentful@7.1.0-beta0)

@emilyemorehouse
Copy link
Member

Yeah, I can work on getting that out. I'm okay with #1718 being listed as a breaking change and moving on as-is, provided no one else is strongly opposed.

@eilgin
Copy link

eilgin commented Nov 7, 2018

More than 2 months later, there's still no news of the 0.19 release. Is this because of #1831? Because it looks like Sauce labs has failed to launched the headless browsers :/

@rolledback
Copy link

rolledback commented Feb 4, 2019

Any update on a 0.19 @emilyemorehouse ? 🙁

@emilyemorehouse
Copy link
Member

I was just looking at why CI is now failing for everything (master was stable and is now failing with only documentation changes, which is suspect). Hoping to get that fixed and cut a release with #1831 if no one is opposed.

@rolledback
Copy link

Awesome! Thank you. 😄

@XiaoningLiu
Copy link

Any update for 0.19.0?

@azizhk
Copy link

azizhk commented Jun 19, 2019

I think this issue can be closed. (Sorry for notification, just small cleanup as I see, to avoid this library from getting abandoned.)

@fed135 fed135 closed this as completed Aug 13, 2019
@axios axios locked and limited conversation to collaborators May 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests