Skip to content

fix(scripts): standardize shebang and clean whitespace in make_docs.sh - #773

Closed
amir-rezaei wants to merge 1 commit into
waltsims:masterfrom
amir-rezaei:fix/make-docs-shebang-and-formatting
Closed

fix(scripts): standardize shebang and clean whitespace in make_docs.sh#773
amir-rezaei wants to merge 1 commit into
waltsims:masterfrom
amir-rezaei:fix/make-docs-shebang-and-formatting

Conversation

@amir-rezaei

@amir-rezaei amir-rezaei commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Description

This PR standardizes the shebang interpreter directive and cleans up leading whitespace in make_docs.sh.

Details

  1. Updated shebang (#! /bin/bash $\to$ #!/usr/bin/env bash).
  2. Removed extraneous leading spaces in shell execution commands.

Greptile Summary

Standardizes the documentation helper script without changing its build behavior.

  • Uses #!/usr/bin/env bash for portable Bash resolution.
  • Removes unnecessary leading whitespace from the Sphinx command and adjacent comment.

Confidence Score: 5/5

The PR appears safe to merge because the documentation command remains unchanged and no repository workflow depends on an incompatible interpreter path.

The script still invokes the same Sphinx build, while the updated shebang uses a conventional portable Bash lookup and existing documentation workflows do not execute this helper.

Important Files Changed

Filename Overview
make_docs.sh The shebang and whitespace cleanup preserve the existing Sphinx documentation build command without introducing a reachable failure.

Reviews (1): Last reviewed commit: "fix(scripts): standardize shebang and cl..." | Re-trigger Greptile

@codecov

codecov Bot commented Jul 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.83%. Comparing base (aff369e) to head (e18a1a5).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #773   +/-   ##
=======================================
  Coverage   75.83%   75.83%           
=======================================
  Files          58       58           
  Lines        8274     8274           
  Branches     1614     1614           
=======================================
  Hits         6275     6275           
  Misses       1373     1373           
  Partials      626      626           
Flag Coverage Δ
3.10 75.80% <ø> (ø)
3.11 75.80% <ø> (ø)
3.12 75.80% <ø> (ø)
3.13 75.80% <ø> (ø)
macos-latest 75.71% <ø> (ø)
ubuntu-latest 75.71% <ø> (ø)
windows-latest 75.64% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@waltsims waltsims closed this Jul 24, 2026
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.

3 participants