Comments (5)
Thanks Mitch,
The change looks simple, but as you've not been able to install your fork on your own system successfully, I am reluctant to accept the PR. I'm unable to test it myself at the moment as I'm travelling over christmas. And once it's accepted I'll also need to push it to NPM -- again, something I can do when i'm home.
Will look at this again when I'm back at home in January. Thanks for investigating it and submitting the PR, in the meantime.
from homebridge-bomgovau.
Oh I just submitted this as a question on reddit and was pointed in the right direction by another developer. @SteveCohen apparently the issue is with node-fetch
"dependencies": {
"node-fetch":">=1.0",
"fakegato-history": ">0.4.0",
"hap-nodejs-community-types": "https://github.com/homespun/hap-nodejs-community-types.git"
}
I was advised you should be using ^
instead of >=
and the version should be taken to 2.6.6
; so all up should be changed to "node-fetch":"^2.6.6"
. I have just created a PR for it if you want to take a look Steve (and make sure I did not do it wrong)?
from homebridge-bomgovau.
Thanks Mitch, that's interesting. I'm learning here too!
I read the reddit thread. Judging by Western_Icy's comment ("esm modules and currently incompatible with homebridge") it sounds like >=1.0
doens't work because it's installing a module that's too new, and not compatible with Homebridge? I'll clarify on Reddit.
Quick question in the meantime. Did you test the PR and confirm it works?
from homebridge-bomgovau.
Hey Steve :) yep definitely still learning myself and only just published my first npm package recently to make my repo available on Homebridge UI.
I actually thought it was the other way around; but I could be wrong! But actually now that I think about it more; based on the ‘>=‘ it should mean that as long as the version is greater than or equal to the designated value then it should be fine. However, I do not understand what the difference is by using ‘^’ but have noted a lot of plugins I am using are using that method in their package.json
dependencies.
I tried to install my fork of your repo through ‘git clone’, ‘npm pack’ and install (something Iv only recently learned to do) for testing but it would not let me install it. Which means I have not been able to test it myself and suggest you test it yourself before approving the PR. I did do the exact same thing for a second plug-in with the exact same ‘node-fetch’ issue; it let me install the fork locally and it worked perfectly. Not sure why your plug-in won’t let me try.
from homebridge-bomgovau.
Hey @SteveCohen, I tired to install my fork of your repo locally and it still will not let me due to NPM error 128; will have to look into it properly later. I am certain it is an issue with my local install of your plugin and not an issue with the changes I have made. Based on the comments from Reddit and the readings you linked there; it looks like the change is required and I will be looking forward to getting this plugin installed again. So would we be good to merge the changes and publish; or is there anything else we need to test first?
from homebridge-bomgovau.
Related Issues (10)
- How to install with NPM? HOT 2
- invalid json response body HOT 4
- Station URL Incorrect HOT 1
- Error on install HOT 2
- Incorrect Plug-in Identifier HOT 2
- Request ability to disable sensors in config HOT 2
- Apply for Homebridge Verified HOT 1
- Use Fakegato HOT 1
- Chance of rain or precipitation actuals HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from homebridge-bomgovau.