Skip to content

Fix CI: run mapper.sh with bash instead of sh#28

Merged
cokelaer merged 1 commit intomainfrom
copilot/fix-ci-switch-sh-to-bash
Apr 4, 2026
Merged

Fix CI: run mapper.sh with bash instead of sh#28
cokelaer merged 1 commit intomainfrom
copilot/fix-ci-switch-sh-to-bash

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 4, 2026

mapper.sh uses set -o pipefail, a bash-specific option not supported by POSIX sh. The test was invoking it with sh, causing all Python version matrix jobs to fail with set: Illegal option -o pipefail.

Change

  • test/test_main.py: replace sh mapper.shbash mapper.sh in test_full

@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 0 complexity · 0 duplication

Metric Results
Complexity 0
Duplication 0

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@coveralls
Copy link
Copy Markdown

coveralls commented Apr 4, 2026

Pull Request Test Coverage Report for Build 23987528449

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 80.0%

Totals Coverage Status
Change from base Build 23363316193: 0.0%
Covered Lines: 56
Relevant Lines: 70

💛 - Coveralls

@cokelaer cokelaer merged commit 0821ee6 into main Apr 4, 2026
4 of 5 checks passed
@cokelaer cokelaer deleted the copilot/fix-ci-switch-sh-to-bash branch April 4, 2026 21:08
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