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

updates #27

Merged
merged 13 commits into from
May 5, 2017
Merged

updates #27

merged 13 commits into from
May 5, 2017

Conversation

ihadeed
Copy link
Contributor

@ihadeed ihadeed commented May 5, 2017

  • removed dependencies
    • made everything a dev dependency to make things simpler
  • added NgModule ( closes Ionic 3 Please add a @NgModule annotation Error #25 )
  • added .editorconfig file
  • removed typings and added @types
  • compiling with ngc instead of tsc to produce needed files for AoT compilation
  • added tests
  • fix linting
  • add circleci instructions

I might add more things to this PR .. still testing the library.. these are just the first things I noticed that need changing

@Nodonisko
Copy link
Owner

Wow! Such a nice PR, I think we should publish this as version 2.0.0

@@ -1,14 +1,5 @@
# Created by .ignore support plugin (hsz.mobi)
Copy link
Owner

Choose a reason for hiding this comment

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

I think this should stay here

@Nodonisko
Copy link
Owner

It looks really nice, please fix .gitignore and we can merge it :)

@ihadeed
Copy link
Contributor Author

ihadeed commented May 5, 2017

Will update it soon, just finishing up the tests.

@ihadeed
Copy link
Contributor Author

ihadeed commented May 5, 2017

Ok @Nodonisko all is done!

I also added CircleCI configuration to the project. You can see a sample build here: https://circleci.com/gh/ihadeed/ionic-cache/1

You just need to sign up there and add your repo so it gets built automatically on pushes/PRs. It's free for open source projects. If you want to use TravisCI instead I can put together a different config file. I personally prefer CircleCI 2.0 since it's much faster. If node_modules is cached, builds take ~17 seconds, see this build for example.

@Nodonisko
Copy link
Owner

It's really awesome. I will merge it and after I will fix few more issues and we can release.

Thanks very much! I will mention you in readme :)

@Nodonisko Nodonisko merged commit 91edc0a into Nodonisko:master May 5, 2017
@ihadeed
Copy link
Contributor Author

ihadeed commented May 5, 2017

It's worth noting that there's one breaking change. I renamed disableCache to enableCache. I thought it would be more intuitive and straightforward.

So to disable cache you need to call:

this.cache.enableCache(false);

and to re-enable you call:

this.cache.enableCache(true);

@Nodonisko
Copy link
Owner

Yes, I think so, it was really bad naming.

But best change is tests, it will help a lot.

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.

Ionic 3 Please add a @NgModule annotation Error
2 participants