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

Integrate another starter in your starter #84

Open
frederikprijck opened this issue Aug 8, 2017 · 10 comments
Open

Integrate another starter in your starter #84

frederikprijck opened this issue Aug 8, 2017 · 10 comments

Comments

@frederikprijck
Copy link
Contributor

frederikprijck commented Aug 8, 2017

Hi,

I've been working on https://github.com/frederikprijck/angularjs-webpack-starter as part of my own setup for a while now.
There's a few differences compared to your starter, any chance you're up for a chat to see if we can integrate some features? I do get there's a lot of starters out there, so if my repo is totally not something you feel comfortable with, I'd totally understand!

Few things I'd prefer in a AngularJS starter these days (as it's all about getting people familiar with the Angular way of doing things to easier shift towards Angular):

  • TypeScript & @types
  • TS Lint
  • Components
  • SCSS
  • Webpack chunks
  • ngAnnotate

I'd be happy to send over a PR to integrate some things if you feel comfortable with it.

Note: I do get it's more interesting to focus on Angular instead of AngularJS these days. However, I think it's not a bad idea to get people more and more familiar with how Angular works (TS, Components, WebPack)

Note2: I'm talking about moving (some of) my things into your repository, not the other way around! :D

@phra
Copy link
Member

phra commented Aug 8, 2017

hi @frederikprijck ,
right now i have no time to upgrade this starter kit but if you want to send a PR i will be very happy to review it and eventually merge it! 🙌

@frederikprijck
Copy link
Contributor Author

frederikprijck commented Aug 8, 2017

@phra Before taking any time to create a PR, it might be good to know if you're interested in going for TS. If not, I can leave out that part (however, I'd encourage using TS), but that's a little bit more rework if you decide to not merge the TS version (which would be somewhat pitty for the time spent :D ).

@phra
Copy link
Member

phra commented Aug 8, 2017

i work most of my time with typescript and i love it.
we can have a separate branch called typescript or merge it in master and move the actual one to a branch called javascript so in any case your work won't be useless.

what is @Foxandxss thinking about this?

@jozzhart
Copy link

jozzhart commented Mar 8, 2018

Hi @phra

Not sure where you got to with this, but I was wondering if you'd be interested in a pull request to add a basic component and karma tests.

@frederikprijck
Copy link
Contributor Author

frederikprijck commented Mar 8, 2018

@jozzhart Not sure how this is related to the current issue (not saying it's not interesting tho, components & tests is interesting for sure! ) .

Never the less I'm still interested to incorporate https://github.com/frederikprijck/angularjs-webpack-starter in this repo. This one is almost on webpack 4 aswell. However, still awaiting feedback.

Using TypeScript will eventually make it easier to integrate with ngUpgrade to eventually move way from AngularJS and use Angular. We've even used my repo to give a talk regarding ngUpgrade and did a live coding session to upgrade the starter setup to Angular (using ngUpgrade) and incorporate it with angular-cli.

I could do a PR and have it reviewed. But I guess it's easier to have a look at my repo and let me know if you're interested in how things are done. Coz it'll change alot of the current setup.

@phra
Copy link
Member

phra commented Mar 12, 2018

if you can work a PR that does:

  1. update deps (update webpack / angular)
  2. update and port to typescript the current components
  3. update the tests

keeping the same UI of starter i would be glad to review the PR.

@cvaliere
Copy link

cvaliere commented Jul 5, 2018

hi @frederikprijck I'm looking for a AngularJS+Webpack+TS+SCSS starter, and the 2 most starred are this one and yours

I've cloned and installed both

this one seems faster, but yours is closer from my final goal

what way should I go?

thanks :)

@frederikprijck
Copy link
Contributor Author

@cvaliere Feel free to report an issue on mine to indicate how you measure the speed.
I'd be happy to fix it.

@cvaliere
Copy link

cvaliere commented Jul 5, 2018

very simply: after starting your projects, I change a file (JS for this one, TS for yours) and I wait for reload
this one is almost immediate to reload, whereas yours takes a few seconds
I'm thinking that maybe it's because yours has 1 more step (compile TS to JS), but I didn't dive deep yet

anyway, as I see that you wanted to merge yours into this one, my question was: do you feel that this one is better for any reason? or should I go with yours (in which case I'd be happy to know what takes so long to reload, if you have any idea abou it)?

@frederikprijck
Copy link
Contributor Author

frederikprijck commented Jul 5, 2018

I prefer mine, as that's why I've created it (it uses TypeScript, Webpack4, ...). But I don't like to pull people towards my repository in another person's repository, that's not done imho. If you want to dicsuss my repository, please open an issue in the repo and I'll be happy to help you / resolve issues.

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

4 participants