Skip to content

implement upload of latest build of Ember.js, this fixes #644 - #659

Merged
wagenet merged 2 commits into
emberjs:masterfrom
pangratz:fix_644
Apr 17, 2012
Merged

wagenet merged 2 commits into
emberjs:masterfrom
pangratz:fix_644

Conversation

@pangratz

@pangratz pangratz commented Apr 6, 2012

Copy link
Copy Markdown
Member

I've tried to code in Ruby and this is the result 😔

It's my try to fix #644. It does work, although I'm sure there is a more Ruby way of coding what I did so please improve my approach.

@wagenet

wagenet commented Apr 6, 2012

Copy link
Copy Markdown
Member

@pangratz Thanks. I'll check this out soon. Since this isn't part of Ember core directly, I'm ok if it's a little rough around the edges. We can always clean it up later.

@pangratz

pangratz commented Apr 6, 2012

Copy link
Copy Markdown
Member Author

@wagenet ok, great!

@pangratz

Copy link
Copy Markdown
Member Author

There is also an issue regarding the downloads with the used github gem, see piotrmurach/github#21

The issue with a failed upload when using the github_api's upload method has been addressed in piotrmurach/github#21
Comment thread Gemfile

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

The nokogiri dependency can be removed as soon as a new version of github_api is released, see piotrmurach/github#23

wagenet added a commit that referenced this pull request Apr 17, 2012
implement upload of latest build of Ember.js, this fixes #644
@wagenet
wagenet merged commit d942038 into emberjs:master Apr 17, 2012
sandstrom pushed a commit to sandstrom/ember.js that referenced this pull request Jun 17, 2021
…rn/bl-4.0.3

[Security] Bump bl from 4.0.2 to 4.0.3
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

"rake upload" broken

2 participants