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

Bank scrapers not updated automatically #13

Open
ifeins opened this issue Feb 8, 2018 · 7 comments
Open

Bank scrapers not updated automatically #13

ifeins opened this issue Feb 8, 2018 · 7 comments

Comments

@ifeins
Copy link
Contributor

ifeins commented Feb 8, 2018

If someone updates the israeli-bank-scrapers npm module then it doesn't automatically gets reflected in the ynab updater because the package.json points to a specific version.

Even if the package.json pointed to the latest version (e.g. israeli-bank-scrapers@latest) then if someone ran npm install a while ago on israeli-ynab-updater then he'll have to remember to run npm install to get the latest bank scrapers.

@eshaham
Copy link
Owner

eshaham commented Feb 10, 2018

@ifeins I think this is how npm updating works.
You need to proactively call npm install after you update current repo from git to make sure all the dependencies are fetched.
^0.5.3 is actually not referencing a specific version, but any minor version following it.
So if at the time you run npm install the dependent module has a higher minor version, say 0.6.1, that version will actually get installed. see here.
Regardless, I don't like using @latest in my dependencies, as it might fetch breaking changes.

@ifeins
Copy link
Contributor Author

ifeins commented Feb 10, 2018

@eshaham Is that true even when package-lock.json has a specific version of israeli-bank-scrapers?

@eshaham
Copy link
Owner

eshaham commented Feb 10, 2018

Actually @ifeins, you are right.
Since this repo has the package lock file checked in, it will force using the same version until someone runs npm update and checks in the update package lock file.
But that should also apply if we had @latest used for a dependency.

@ifeins
Copy link
Contributor Author

ifeins commented Feb 12, 2018

@eshaham Got it, I'll put some more thought into it and see if somethings comes up.

@eshaham
Copy link
Owner

eshaham commented Feb 20, 2018

@ifeins let me know if we can close this issue

@ifeins
Copy link
Contributor Author

ifeins commented Feb 22, 2018

@eshaham It's still an issue, don't have any solve yet.

@esakal
Copy link
Contributor

esakal commented Apr 12, 2018

@eshaham are you planning to deploy this app as a node standalone application?

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

3 participants