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

Added function keyword #2

Merged
merged 1 commit into from
Sep 25, 2015
Merged

Conversation

vitch
Copy link
Contributor

@vitch vitch commented Sep 25, 2015

Trying to install this plugin from github (ember install ember-cli-deploy/ember-cli-deploy-s3-index) results in an error:

Unexpected token (

This fixes it and I presume is intended behaviour unless you are somehow running npm through babel? I'm on node v0.12.7

Trying to install this plugin from github (`ember install ember-cli-deploy/ember-cli-deploy-s3-index`) results in an error:

> Unexpected token (

This fixes it and I presume is intended behaviour unless you are somehow running npm through babel? I'm on node `v0.12.7`
@LevelbossMike
Copy link
Member

Ah yes of course. I was developing this with iojs. Sorry for the inconvenience caused and thx for your pr. I'll add tests to the repo this weekend and release the first beta version

LevelbossMike added a commit that referenced this pull request Sep 25, 2015
Added function keyword
@LevelbossMike LevelbossMike merged commit 1937fe8 into ember-cli-deploy:master Sep 25, 2015
@vitch vitch deleted the patch-1 branch September 25, 2015 14:39
@vitch
Copy link
Contributor Author

vitch commented Sep 25, 2015

Thanks!

Side question - should it work as is (with these fixes in place)?

It now doesn't error for me but no file is uploaded to my bucket and I'm not sure if it's something I'm doing or something missing from the plugin (I'm also using ember-cli-deploy for the first time)

@LevelbossMike
Copy link
Member

I am using this right now in a test application and it works fine. what does your deploy.js look like?

@vitch
Copy link
Contributor Author

vitch commented Sep 25, 2015

Sorry - PBKAC!

I had the wrong bucket name in there (it was uploading to the assets bucket instead of the separate bucket we have set up for the index).

Thanks!

@LevelbossMike
Copy link
Member

ok great to hear! Please let me know of any issues you run into. I'll release the first beta this weekend so just ping me if you run into problems so we can sort them out asap

@vitch
Copy link
Contributor Author

vitch commented Sep 25, 2015

Will do - thanks 😃

vitch pushed a commit to vitch/ember-cli-deploy-s3-index that referenced this pull request Mar 23, 2017
…sync-aws-dotenv-with-boto to master

* commit 'd8f88377bec1dece65d1a368ee1479e16cc5b0e3':
  Use Boto's env var name, rather than our own
vitch pushed a commit to vitch/ember-cli-deploy-s3-index that referenced this pull request Mar 23, 2017
…g from feature/cs-18036-active-revision to master

* commit '84c78fdc9e0b9fb601f178489c7138fec63a077c':
  CS-18036 - Addressing CR feedback
  Linting fixes
  CS-18036 - Add Author Date and Commiter Date to the returns commit json
  CS-18036 - Create an active-revision.json file on didActivate so we can easily display the active version from swift-ui list
  CS-18036 - Bugfix: check for when no intital revisions are availble
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants