Skip to content

feat: add ENABLE_REPORT_ISSUES env var to control issue creation#31

Merged
zkoppert merged 1 commit intomainfrom
feat/enable-report-issues
Mar 14, 2026
Merged

feat: add ENABLE_REPORT_ISSUES env var to control issue creation#31
zkoppert merged 1 commit intomainfrom
feat/enable-report-issues

Conversation

@zkoppert
Copy link
Contributor

@zkoppert zkoppert commented Mar 14, 2026

Closes #28

Changes

  • Add ENABLE_REPORT_ISSUES env var to env.py (default: true for backward compatibility)
  • Guard issue creation in pr_conflict_detector.py with the new flag
  • Add tests for the new env var and behavior
  • Update README.md documentation

Usage

env:
  ENABLE_PR_COMMENTS: "true"
  ENABLE_REPORT_ISSUES: "false"
  ENABLE_GITHUB_ACTIONS_STEP_SUMMARY: "true"

Testing

  • make lint - all 5 linters pass (flake8, isort, pylint 10.00/10, mypy, black)
  • make test - 206 tests pass, 99.17% coverage (well above 80% threshold)
  • ✅ Local integration test with ENABLE_REPORT_ISSUES=false - script ran against github/new-user-experience with DRY_RUN=true. Output confirmed: "Report issue creation disabled (ENABLE_REPORT_ISSUES=false)" and no issue creation was attempted.
  • ✅ Local integration test with ENABLE_REPORT_ISSUES=true (default) - script ran successfully and proceeded to issue creation path, correctly stopped by dry run: "DRY RUN: Skipping issue creation". Confirms default behavior is preserved.

Add a new ENABLE_REPORT_ISSUES environment variable (default: true)
that controls whether conflict report issues are created/updated in
target repositories. This allows users to get PR comments and step
summaries without the report issues.

Closes #28

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions github-actions bot added the feature New feature or functionality label Mar 14, 2026
@zkoppert zkoppert self-assigned this Mar 14, 2026
@zkoppert zkoppert marked this pull request as ready for review March 14, 2026 08:08
@zkoppert zkoppert requested a review from jmeridth as a code owner March 14, 2026 08:08
@zkoppert zkoppert merged commit caee201 into main Mar 14, 2026
40 checks passed
@zkoppert zkoppert deleted the feat/enable-report-issues branch March 14, 2026 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: add ENABLE_REPORT_ISSUES env var to control issue creation

2 participants