Skip to content

ci: validate .env.example by copying it before test runs#617

Merged
Chemaclass merged 12 commits intomainfrom
fix/env-example-comment-spacing
Mar 30, 2026
Merged

ci: validate .env.example by copying it before test runs#617
Chemaclass merged 12 commits intomainfrom
fix/env-example-comment-spacing

Conversation

@antonio-gg-dev
Copy link
Copy Markdown
Member

@antonio-gg-dev antonio-gg-dev commented Mar 30, 2026

Related #

make test loads .env via the Makefile, but CI test workflows were not creating that file from .env.example. That meant a syntax error in .env.example could go unnoticed in GitHub Actions while still breaking local test runs for contributors following the setup documented in CONTRIBUTING.md.

image

Changes

  • Fixed the malformed BASHUNIT_COVERAGE_THRESHOLD_HIGH entry in .env.example by separating the assignment from the inline comment so Bash can parse it correctly.
  • Updated the test workflows to copy .env.example to .env before running the suite, so CI now exercises the same configuration path used locally.
  • This prevents regressions where .env.example drifts into an invalid Bash file without being caught by automated checks.

Checklist

  • I updated the CHANGELOG.md to reflect the new feature or fix
  • I updated the documentation to reflect the changes

@github-actions
Copy link
Copy Markdown
Contributor

✅ Contributor Report

User: @antonio-gg-dev
Status: Passed (12/13 metrics passed)

Metric Description Value Threshold Status
PR Merge Rate PRs merged vs closed 100% >= 30%
Repo Quality Repos with ≥100 stars 1 >= 0
Positive Reactions Positive reactions received 83 >= 1
Negative Reactions Negative reactions received 3 <= 5
Account Age GitHub account age 3894 days >= 30 days
Activity Consistency Regular activity over time 100% >= 0%
Issue Engagement Issues with community engagement 0 >= 0
Code Reviews Code reviews given to others 6 >= 0
Merger Diversity Unique maintainers who merged PRs 1 >= 2
Repo History Merge Rate Merge rate in this repo 100% >= 0%
Repo History Min PRs Previous PRs in this repo 4 >= 0
Profile Completeness Profile richness (bio, followers) 70 >= 0
Suspicious Patterns Spam-like activity detection 0 N/A

Contributor Report evaluates based on public GitHub activity. Analysis period: 2025-03-30 to 2026-03-30

antonio-gg-dev and others added 3 commits March 30, 2026 13:15
The .env file sourcing overwrites BASHUNIT_BOOTSTRAP_ARGS and
BASHUNIT_NO_COLOR set by the test environment or early flag scan.
@Chemaclass Chemaclass added the bug Something isn't working label Mar 30, 2026
@Chemaclass Chemaclass force-pushed the fix/env-example-comment-spacing branch 2 times, most recently from 25d3152 to dc0d7e1 Compare March 30, 2026 14:05
@Chemaclass Chemaclass force-pushed the fix/env-example-comment-spacing branch from dc0d7e1 to 627e6b4 Compare March 30, 2026 14:17
@Chemaclass Chemaclass merged commit 73a5a5e into main Mar 30, 2026
29 checks passed
@Chemaclass Chemaclass deleted the fix/env-example-comment-spacing branch March 30, 2026 15:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants