Skip to content

Commit 893088f

Browse files
jfrench9claude
andauthored
refactor: standardize shell scripts with .sh extension (#54)
## Summary This PR standardizes our shell script naming convention by adding the `.sh` extension to all script files in the bin directory and updating the corresponding references in the justfile. ## Key Accomplishments - **Improved Script Identification**: All shell scripts now have explicit `.sh` extensions, making them immediately recognizable as shell scripts - **Enhanced Developer Experience**: Scripts are now easier to identify and work with in IDEs and text editors that rely on file extensions for syntax highlighting and tooling - **Consistent Naming Convention**: Establishes a clear standard for future shell script additions to the project - **Updated Build Configuration**: All script references in the justfile have been updated to reflect the new naming convention ## Files Affected - Renamed 4 shell scripts to include `.sh` extension - Updated justfile to reference the renamed scripts ## Breaking Changes ⚠️ **Potential Impact**: Any external tools, documentation, or automation that directly references the old script names will need to be updated to use the new `.sh` extensions. ## Testing Notes - Verify that all just commands continue to function correctly - Ensure script execution permissions are preserved after the rename - Test that any CI/CD pipelines or external integrations still work as expected ## Infrastructure Considerations This change improves maintainability and follows common shell scripting conventions. The explicit file extensions will help with tooling integration and make the codebase more accessible to new contributors. --- 🤖 Generated with [Claude Code](https://claude.ai/code) **Branch Info:** - Source: `refactor/rename-shell-files` - Target: `main` - Type: feature Co-Authored-By: Claude <noreply@anthropic.com>
2 parents ee62b60 + 6301065 commit 893088f

5 files changed

Lines changed: 6 additions & 6 deletions

File tree

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ set -e
33

44
# Create feature branch script - local Git operations
55
# Creates a new feature/bugfix/hotfix branch locally and pushes to remote
6-
# Usage: ./bin/create-feature [feature|bugfix|hotfix|chore|refactor] [branch-name] [base-branch] [update-deps]
6+
# Usage: ./bin/create-feature.sh [feature|bugfix|hotfix|chore|refactor] [branch-name] [base-branch] [update-deps]
77

88
# Default values
99
BRANCH_TYPE=${1:-feature}
File renamed without changes.
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ set -e
33

44
# Create release script using GitHub Actions
55
# Creates a new release branch via GHA workflow and checks it out locally
6-
# Usage: ./bin/create-release [major|minor|patch]
6+
# Usage: ./bin/create-release.sh [major|minor|patch]
77

88
# Default to patch if no argument provided
99
VERSION_TYPE=${1:-patch}
File renamed without changes.

justfile

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -45,23 +45,23 @@ typecheck:
4545

4646
# Generate SDK from localhost API
4747
generate-sdk url="http://localhost:8000/openapi.json":
48-
bin/generate-sdk {{url}}
48+
bin/generate-sdk.sh {{url}}
4949

5050
# Build python package locally (for testing)
5151
build-package:
5252
python -m build
5353

5454
# Create a feature branch
5555
create-feature branch_type="feature" branch_name="" base_branch="main" update="yes":
56-
bin/create-feature {{branch_type}} {{branch_name}} {{base_branch}} {{update}}
56+
bin/create-feature.sh {{branch_type}} {{branch_name}} {{base_branch}} {{update}}
5757

5858
# Version management
5959
create-release type="patch":
60-
bin/create-release {{type}}
60+
bin/create-release.sh {{type}}
6161

6262
# Create PR
6363
create-pr target_branch="main" claude_review="true":
64-
bin/create-pr {{target_branch}} {{claude_review}}
64+
bin/create-pr.sh {{target_branch}} {{claude_review}}
6565

6666
# Clean up development artifacts
6767
clean:

0 commit comments

Comments
 (0)