Repository navigation
feat(bin): add photo and video uploaders - #31
Conversation
Quote expansions in gifme, kubectl-ssh, and suitup, loop over project roots in suitup, and disable SC1071 so zsh scripts stop erroring in the LSP. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
.shellcheckrc is not found when an LSP lints via stdin from another directory. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Switch brewup, daily, dbr, git-adopt, set_ip, and screenshots to bash so shellcheck covers them, fix the findings, and syntax check the remaining zsh scripts with zsh -n in CI. CI now only lints shell files and fails on findings. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
photo posts images to photos.natwelch.com, prints the URLs, and copies them to the clipboard. gifme now builds with magick and uploads through photo instead of CloudApp. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
# Conflicts: # bin/gifme
There was a problem hiding this comment.
🟡 Changes recommended
Multipart filename handling can cause valid image paths to fail or upload unintended files.
Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.
video posts to videos.natwelch.com and prints the watch URLs. Both uploaders quote filenames so commas and semicolons reach the server intact, and print the server's error message on failure. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
icco
changed the title
|
Fixed in b5c65c1: filenames are now quoted in the curl form field, so commas and semicolons upload intact. |
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.
There was a problem hiding this comment.
Copilot wasn't able to review any files in this pull request. Check if the Files changed in this pull request are included in default exclusions.
Why
Adds
bin/photoandbin/video, which upload to photos.natwelch.com and videos.natwelch.com, print the URLs, and copy them to the clipboard.gifmenow builds withmagickand uploads throughphotoinstead of CloudApp.Risks
Both need
jq.videopredicts the watch URL from the job id and the current UTC year, and it 404s until conversion finishes.