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

release script simplifications #341

Merged
merged 3 commits into from
Mar 21, 2024

Conversation

whot
Copy link
Member

@whot whot commented Mar 11, 2024

two cleanups and one simplification: we don't need to enforce the extra dot for the module: release.sh .. We only ever use this script on one repo anyway, so let's imply that.

whot added 3 commits March 11, 2024 10:45
Might as well since we only ever release one module at a time with this
script.
No longer hooked into anything so let's not pretend we have it.
These haven't been needed in decades. Where a string may be empty use
quotes, otherwise let's switch string comparisons to just that. And
where we check for empty/nonempty use -z/-n.
@Pinglinux
Copy link
Member

Tested-by: Ping Cheng [email protected]

It worked for me. Thank you Peter!

@whot whot merged commit f424151 into linuxwacom:master Mar 21, 2024
13 checks passed
@whot
Copy link
Member Author

whot commented Mar 21, 2024

thanks!

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