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

Pull Request Question, ES6 Rewrite #23

Open
billrawlinson opened this issue Oct 6, 2015 · 4 comments
Open

Pull Request Question, ES6 Rewrite #23

billrawlinson opened this issue Oct 6, 2015 · 4 comments

Comments

@billrawlinson
Copy link

this is a question more than an issue.

I forked this and rewrote it in ES6 and made it so there isn't a global dependancy on lodash (still uses lodash at moment but you don't need to include it globally). I also extracted the template to it's own file and export the service as part of the module

I use jspm to install it and I use gulp (instead of grunt) to build it.

I wasn't sure if you'd want such a massive change as a pull request so I figured I'd ask you and you can check it out and let me know.

https://github.com/billrawlinson/ng-password-strength

@billrawlinson billrawlinson changed the title Pull Request Question Pull Request Question, ES6 Rewrite Oct 6, 2015
@subarroca
Copy link
Owner

Some of the pending pull requests already do all you're asking for. In fact the only thing missing is the es6 rewriting. We could unify both modules but keep es5 and es6 detached for now. What do you think?

@billrawlinson
Copy link
Author

I am not sure how you'd manage that kind of unification. If you have a plan for doing so I certainly have no problem with it.

@subarroca
Copy link
Owner

I just published a new version, 0.3.0. I've defined a structure and a naming for factories. You can take a look at it and use the same structure and naming. Or if you think there's a better one just say so and I'll make the changes

@billrawlinson
Copy link
Author

I'll have to take some time this weekend to dig into your changes. Thanks for the heads up.

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

No branches or pull requests

2 participants