-
Notifications
You must be signed in to change notification settings - Fork 135
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
Roadmap #38
Comments
@marcmeans - FYI |
Linting tslint should work out of the box if you have a tslint.json. If you would like further enhancements to linting please raise an issue and explain there. |
Testing with watches Was implemented in #19. Explained in the blog and readme. If you would like further enhancements to this please raise an issue and explain there. |
examples of tests mocking observables/promises I don't want this repo to turn into a "how to unit test" thing. It's a rabbit hole and I don't have the time or inclination to support it. That being said there are certain things that I think are worth including, for example, testing Ionic @Pages using angular2 dependency injection, which is not done yet - though components are. The best place for general unit testing questions probably stack overflow. There is an example with promises in clickers.spec.ts (most of the complex examples are therein). |
put example code in module structure for easy removal and showing pattern @marcmeans - sounds interesting, but I don't know what you mean? Maybe raise an issue with some explanation? |
What about Hot Module Replacement? |
@geoHeil are you suggesting a move back to webpack (wouldn't happen unless Ionic went back again), or using browserify-hmr? or something else? |
Not sure about implementation details, just saying HMR would be cool. As such |
agreed |
In this post, seems Ionic team are migrating again from Browserify to Webpack 2 to take advantage of "tree shaking." .. It would be nice start moving this project to webpack.. what do you think ? |
Yeah, if they do end up going with webpack2 in beta12 we'll switch over as I don't think it's worth doing ahead of time. On 13 Aug 2016 02:44, "Damien" [email protected] wrote: In this post http://blog.ionic.io/announcing-ionic-2-beta-11/, seems — |
Would be good to have same code documentation, maybe through typedoc ? |
@marcoturi - agreed. I've opened #139. If you're feeling generous, a small PR would be fantastic. If not thanks a lot for the suggestion, will do my best get round to it! |
@lathonez Have you put any thought into doing e2e testing on actual devices or through an emulator? |
#52 (top of the list). Last time I looked SauceLabs was the best option for FOSS. Realistically I don't have any time for moving this roadmap forward (until my associated closed source project requires it). Very happy for PRs / discussion suggestions around the correct direction. |
Ah it was hidden in the name. Yeah I'm currently checking out both a SauceLabs solution and Appium. |
If you come up with something that works let us know, maybe share config. Thanks |
Now we're using @angular/cli, anything in the test direction should be raised upstream there. We'll take any features they add. |
Open:
Closed:
implement ngrximplement HMRdynamic themingsonar qube analysise2e coveragelintingtesting with watchesRemove module loader dependency and requirement for wrapping tests in maintypescript maps for debug/coverageexamples of tests mocking observables/promisesput example code in module structure for easy removal and showing patternbuild location for custom images usede2e testing with protractorupgrade travis to trusty / containerisedremove app.stub.jsunit test Ionic Pages using Angular2 DIcentralise Testing DI Boilerplateintegrate with appveyorTo propose a feature for the roadmap please raise an enhancement issue!
The text was updated successfully, but these errors were encountered: