Skip to content
This repository has been archived by the owner on Nov 25, 2020. It is now read-only.

Dep update aurelia-rc1 #204

Merged
merged 6 commits into from
Jul 12, 2016
Merged

Dep update aurelia-rc1 #204

merged 6 commits into from
Jul 12, 2016

Conversation

doktordirk
Copy link
Contributor

  • dependencies update to aurelia rc1
  • build task copied from aurelia and adapted

@doktordirk
Copy link
Contributor Author

not working yet for aurelia-cli (pathes of jwr-decode not resolved correctly) and new webpack-plugin (wierd DI issue)

.pipe(gulp.dest(paths.output + moduleType));
});

if (moduleType === 'native-modules') return; // typescript doesn't support the combination of: es5 + native modules
Copy link

@icanhasjonas icanhasjonas Jul 6, 2016

Choose a reason for hiding this comment

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

It's a recently added feature in TypeScript 2.0 (currently in beta)
microsoft/TypeScript#6319

Copy link
Contributor Author

Choose a reason for hiding this comment

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

oh, i just copy the build from aurelia. but if you can help with the webpack and cli issues ... 😄

Copy link
Member

Choose a reason for hiding this comment

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

Curly brace

Copy link
Contributor Author

Choose a reason for hiding this comment

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

that's just copied from aurelia. seems easier not change as few as possible to make updating easier

@doktordirk
Copy link
Contributor Author

ps: atm can be tested with eg jspm i aurelia-authentication=github:doktordirk/aurelia-authentication@dep-update or npm i github:doktordirk/aurelia-authentication#dep-update (i think)

@RWOverdijk
Copy link
Member

Looks... Okay. I haven't tested it yet but if anyone else could that'd be cool.

@doktordirk doktordirk merged commit 928a1f9 into SpoonX:master Jul 12, 2016
@doktordirk doktordirk deleted the dep-update branch July 12, 2016 12:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants