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

examples: Improve README by describing scripts #2976

Merged
merged 1 commit into from
Jun 21, 2024

Conversation

gdt
Copy link
Collaborator

@gdt gdt commented Jun 20, 2024

Expand description of the directory, making it clearer that some programs have production use.

Declare that python scripts are python3. It's clear that nobody is trying to use them with python 2.7, and if they did, we would tell them not to.

List the scripts with a one-line description, sorting them into production (none yet), generally usable, and true examples. Leave some uncategorized.

Expand description of the directory, making it clearer that some
programs have production use.

Declare that python scripts are python3.  It's clear that nobody is
trying to use them with python 2.7, and if they did, we would tell
them not to.

List the scripts with a one-line description, sorting them into
production (none yet), generally usable, and true examples.  Leave
some uncategorized.
@zuckschwerdt
Copy link
Collaborator

Looks good! You can delete the sigrok-* entries. We once had external scripts for conversion but rtl_433 can now handle that internally. We should delete the deprecated scripts.

@gdt gdt merged commit 432edcd into merbanan:master Jun 21, 2024
8 checks passed
@gdt gdt deleted the feature.examples-readme branch June 21, 2024 19:35
gdt added a commit to gdt/rtl_433 that referenced this pull request Jun 21, 2024
Per a comment in merbanan#2976 these are no longer necessary.
zuckschwerdt pushed a commit that referenced this pull request Jun 21, 2024
Per a comment in #2976 these are no longer necessary.
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