-
Notifications
You must be signed in to change notification settings - Fork 26
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
Could you publish fanuc-driver
on nuget.org?
#40
Comments
Would I be versioning the entire codebase? |
Well, I’d version However, now that I think about it, IMHO you created collectors/strategies/vendors/handlers (or at least some of them) as they could/should be configurable by Actually, even I suggest to publish it on nuget.org so that we could simply install it without manually building it (unless we want to modify some code). Versioning would help us install a specific version (or even latest stable, dev, rc). As I see it,
Note that is how I’d like |
I agree on all of the above. |
It should be used as an external library/package, but IMHO it should be utilised by the microservice (not |
I was thinking about the different consumers of the data. Some people will want it in SQL, some will want an HTTP endpoint, some will want something else. I think MQTT, although external to the adapter, should be the primary mode of transmission of the data from fanuc-driver to another process.... unless a pattern is defined for 'output plugins'. I reviewed the concepts of https://docs.ladder99.com/en/drivers/base-driver I guess I meant for the Even looking at the latest https://github.com/Ladder99/fanuc-driver/blob/main/fanuc/handlers/ThreeWay.cs So maybe the MQTT configuration should be moved the handler section in YML like here. This way, dependency on MQTT would be removed from the driver. |
|
https://www.lostindetails.com/articles/Native-Bindings-in-CSharp I was looking at moving the Focas DLLs to a subfolder. Will need to test adding a search path on Linux. |
It might be a good idea to publish
fanuc-driver
on nuget.org for easier installation.At the same time, you might want to create GitHub Releases too. 😉
The text was updated successfully, but these errors were encountered: