Skip to content

🧼 [just] gh-process v3.9 fixes shellcheck warnings#15

Merged
chicks-net merged 1 commit intomainfrom
chicks/2025-11-18-gh-process-3-9
Nov 18, 2025
Merged

🧼 [just] gh-process v3.9 fixes shellcheck warnings#15
chicks-net merged 1 commit intomainfrom
chicks/2025-11-18-gh-process-3-9

Conversation

@chicks-net
Copy link
Copy Markdown
Member

Done

  • 🧼 [just] gh-process v3.9 fixes shellcheck warnings

Meta

(Automated in .just/gh-process.just.)

Copilot AI review requested due to automatic review settings November 18, 2025 14:25
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the .just/gh-process.just file to address shellcheck warnings by improving shell script best practices. The changes focus on proper quoting of variables, replacing backticks with $() syntax, and expanding git command aliases to their full forms for better portability and clarity.

Key Changes:

  • Replaced deprecated backtick command substitution with modern $() syntax
  • Added proper quoting around variable expansions to prevent word splitting
  • Expanded git aliases (stp, pushup, co) to their full commands with inline comments

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@chicks-net
Copy link
Copy Markdown
Member Author

PR review was broken by the current cloudflare outage.

@chicks-net chicks-net merged commit 7f6a0c4 into main Nov 18, 2025
1 check failed
@chicks-net chicks-net deleted the chicks/2025-11-18-gh-process-3-9 branch November 18, 2025 14:32
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