-
Notifications
You must be signed in to change notification settings - Fork 5
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
Remove C# Cloud Connector and Restructure cloud_connectors
directory
#58
Conversation
Alex Recommends ReportAlex recommends the following language changes, but Alex is a regular expression based algorithm, so take them with a grain of salt. ✨ 🚀 ✨ Nothing to Report ✨ 🚀 ✨ |
cloud_connectors
directory
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Several minor comments. Please take a look.
cloud_connectors/resources/scripts/digital_twins_setup_smart_trailer.sh
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approved
Removes the C# cloud connector in favor of the MQTT Cloud Connector. The C# Connector was an interim sample intended for a basic demo scenario, but the MQTT sample is now preferred for end-to-end sample scenarios. This PR consists of the following changes:
cloud_connectors/azure/README.md
,cloud_connectors/azure/sample_dtdl/README.md
, andcloud_connectors/azure/mqtt_connector/README.md
file into thecloud_connectors/mqtt_connector/README.md
filecloud_connectors
directory so that it's organized more intuitively now that the above content is removed:azure
directory from the hierarchy since the connector is not Azure-specificresources
directory instead of being nested inside the MQTT connector source codeazure
directory into theresources
directory. This includes scripts and sample DTDL files.sample-dtdl
directory tosample_dtdl
to be consistent about directory naming conventions