FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Upgrade Thrift to ~0.9.1 by agnoster · Pull Request #4 · Applifier/node-scribe · GitHub

This repository was archived by the owner on Jul 28, 2020. It is now read-only.
/ node-scribe Public archive

Upgrade Thrift to ~0.9.1 - #4

Open
agnoster wants to merge 2 commits into
Applifier:masterfrom
AsanaArchive:thrift-upgrade
ConversationCommits (2)ChecksFiles changed
Open

Upgrade Thrift to ~0.9.1#4
agnoster wants to merge 2 commits into
Applifier:masterfrom
AsanaArchive:thrift-upgrade

Conversation

agnoster commented Dec 18, 2013

Copy link
Copy Markdown

This fixes a unicode issue

Upgrade to latest Thrift, ~0.9.1
97fa446
Merge remote-tracking branch 'upstream/master' into thrift-upgrade
1bbb517 
* upstream/master:
  Bump npm version
  Change request unique id random generator to a better implementation

jorgemsrs commented Mar 28, 2015

Copy link
Copy Markdown

This also provides iojs compatibility.

Is there a chance this pull request gets accepted?

suryatech commented Apr 20, 2015

Copy link
Copy Markdown

Latest stable version of thrift now would be 0.9.2

@orktes Any updates on this?

garo commented Apr 22, 2015

Copy link
Copy Markdown
Contributor

Thanks for your contributions and sorry for the way too long time to respond.

Due to the fact that our long term plan is to deprecate scribe usage inside our own infrastructure we haven't had time to look on this pull request. I will most probably merge this soon and bump minor version number to 0.1.0

CatTail commented Oct 9, 2015

Copy link
Copy Markdown

Please update thrift version, current version have issues deal with CJK string.

wision commented Jan 4, 2016

Copy link
Copy Markdown

@garo Any news on this?

springuper commented Feb 24, 2016

Copy link
Copy Markdown

0.9.2 is more reasonable for some bugfixs, hope to update

HugoMFernandes commented Jul 4, 2016

Copy link
Copy Markdown

@garo Any news on this?

rooftopsparrow commented Jan 6, 2017

Copy link
Copy Markdown

Bump

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

9 participants

Back | FazBrowse Home | New Git URL