Skip to content
This repository has been archived by the owner on May 23, 2019. It is now read-only.

fix #154 values for 0 #156

Merged
merged 1 commit into from
Jul 1, 2017
Merged

fix #154 values for 0 #156

merged 1 commit into from
Jul 1, 2017

Conversation

cwtuan
Copy link
Contributor

@cwtuan cwtuan commented Apr 13, 2017

No description provided.

@scriby
Copy link

scriby commented May 25, 2017

Also fixes #88, #112, #119, #127, #136, #141, #144, and an untold number of future issues caused by developers making a perfectly reasonable assumption about the behavior of this library.

(The assumption I'm referring to is that if a developer has observed that all non-zero numbers get coerced to a string during string formatting, that zero would also get coerced to a string.)

@jasonmit
Copy link
Collaborator

@caridy any chance we can land this?

@cwtuan
Copy link
Contributor Author

cwtuan commented Jun 7, 2017

could anyone accept the PR?

@cwtuan
Copy link
Contributor Author

cwtuan commented Jun 15, 2017

@jasonmit Could I take the ownership too? I'm willing to maintain this project

@steida
Copy link

steida commented Jun 21, 2017

@ericf How we can help you with PR?

@okuryu
Copy link
Member

okuryu commented Jun 22, 2017

Could you add tests for this?

@steida
Copy link

steida commented Jun 22, 2017

@okuryu Doing OSS 24/7 already :-/

@rlivsey
Copy link

rlivsey commented Jun 22, 2017

@cwtuan feel free to pull in the tests I wrote for #112

@MaKleSoft
Copy link

How is this not merged after over two months? It's a straightforward fix for a very prominent problem! Can somebody please just hit that merge button?

@steida
Copy link

steida commented Jun 30, 2017

I always wonder what terrible had to happen in maintainers life that they even don't bother to answer or kill the project with some short explanation. And yet, I am open source maintainer too.

@steida
Copy link

steida commented Jun 30, 2017

I see, caridy@gmail.com doesn't work for Yahoo anymore.

Copy link
Collaborator

@caridy caridy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems to be very straight forward, but probably a non-backward compatible change. I think we will have to wait for a Yahoo maintainer to chime in and release a new version of this with this change in. Unfortunately @ericf is not longer at Yahoo, and I don't know who is maintaining this anymore. I certainly can't do the release anymore, so my approve here means little :/

@steida
Copy link

steida commented Jul 1, 2017

I was chatting with former Yahoo developers, it seems nobody in Yahoo is interested in maintenance. We have to fork it probably.

@okuryu
Copy link
Member

okuryu commented Jul 1, 2017

Last month I got commit bits of all repositories of FormatJS suite including intl-messageformat, so I definitely would like to cooperate with these maintenance.

@MaKleSoft
Copy link

Does this mean https://formatjs.io/ is dead?

@steida
Copy link

steida commented Jul 1, 2017

Probably is. It has all symptoms. Key people left the company, and the company doesn't care about open source at all.

@jasonmit jasonmit merged commit 33fddfa into formatjs:master Jul 1, 2017
@jasonmit
Copy link
Collaborator

jasonmit commented Jul 1, 2017

I reverted the commit - did not see the discussion that took place since I last had the tab opened. I reopened with @rlivsey's tests added and took into account past comments by @ericf to guard against false.

#162

@okuryu @caridy can you merge and version a new major?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants