Skip to content

Use python3 explicitly in shell scripts - #10

Merged
pmca31 merged 2 commits into
build-octofit-appfrom
copilot/sub-pr-2-c0f5406f-183c-43a3-8edc-1fe3ffaf6ce0
Feb 11, 2026
Merged

Use python3 explicitly in shell scripts#10
pmca31 merged 2 commits into
build-octofit-appfrom
copilot/sub-pr-2-c0f5406f-183c-43a3-8edc-1fe3ffaf6ce0

Conversation

Copilot AI commented Feb 11, 2026

Copy link
Copy Markdown
Contributor

Addresses feedback from PR #2 to use python3 instead of python in shell scripts, ensuring Python 3 is invoked rather than potentially resolving to Python 2 or a system interpreter.

Changes

  • verify_setup.sh: Changed heredoc from python << EOF to python3 << EOF
  • test-teams-api.sh: Changed JSON tool invocations from python -m json.tool to python3 -m json.tool

Aligns with existing pattern in test_api.sh which already uses python3.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: pmca31 <6774003+pmca31@users.noreply.github.com>
Copilot AI changed the title [WIP] Update registration validation based on feedback Use python3 explicitly in shell scripts Feb 11, 2026
Copilot AI requested a review from pmca31 February 11, 2026 19:30
@pmca31
pmca31 marked this pull request as ready for review February 11, 2026 19:48
@pmca31
pmca31 merged commit dbea9cd into build-octofit-app Feb 11, 2026
@pmca31
pmca31 deleted the copilot/sub-pr-2-c0f5406f-183c-43a3-8edc-1fe3ffaf6ce0 branch February 11, 2026 19:48
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