Skip to content

case of script name and actually usable version number#9

Open
jonorthwash wants to merge 1 commit into
weewx-mqtt:masterfrom
jonorthwash:patch-1
Open

case of script name and actually usable version number#9
jonorthwash wants to merge 1 commit into
weewx-mqtt:masterfrom
jonorthwash:patch-1

Conversation

@jonorthwash
Copy link
Copy Markdown

No description provided.

@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@bellrichm
Copy link
Copy Markdown
Contributor

Thank you for your contribution. A couple of comments.

re: MQTTSubscribe.py vs mqttsubscribe.py
Unfortunately you got caught in me trying to migrate from MQTTSubscribe.py to mqttsubscribe.py. The README and latest (unreleased) code expect mqttsubscribe.py, but the released code expects MQTTSubscribe.py.
I just committed a shim to the unreleased code so that MQTTSubscribe.py will work. I will roll-back all references to MQTTSubscribe.py in the README.

re: Version example
I'm not sure what I want to do here. I really don't want to have to update it every time a new release is out. I am thinking that truly invalid release (won't even download) might be the best value.....

bellrichm added a commit that referenced this pull request Apr 24, 2026
Comment thread docs/readme.md
@@ -134,7 +134,7 @@ WEECTL=$WEEWX_REPO/bin/weectl
For example, if the desired version is 2.3.1.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This version number would also need to change

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.

2 participants