Skip to content

Skip test_bad_commit online variant on CI#1676

Merged
nathanjmcdougall merged 2 commits intomainfrom
copilot/dont-run-test-bad-commit-online
Mar 31, 2026
Merged

Skip test_bad_commit online variant on CI#1676
nathanjmcdougall merged 2 commits intomainfrom
copilot/dont-run-test-bad-commit-online

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 31, 2026

test_bad_commit accesses sync-with-uv on GitHub during the ONLINE parametrized variant, causing flaky CI failures. Skip the online variant when running on CI; locally both variants still run.

  • Skip when CI env var is set and usethis_config.offline is False (i.e. the ONLINE variant of _vary_network_conn)
if os.environ.get("CI") and not usethis_config.offline:
    pytest.skip("Skipping online test on CI to avoid flakiness")

Copilot AI changed the title [WIP] Fix CI to run test_bad_commit test offline only Skip test_bad_commit online variant on CI Mar 31, 2026
Copilot AI requested a review from nathanjmcdougall March 31, 2026 22:41
@nathanjmcdougall nathanjmcdougall marked this pull request as ready for review March 31, 2026 23:48
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 31, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq bot commented Mar 31, 2026

Merging this PR will not alter performance

✅ 2 untouched benchmarks


Comparing copilot/dont-run-test-bad-commit-online (88abdc0) with main (6a813a7)

Open in CodSpeed

@nathanjmcdougall nathanjmcdougall merged commit ea64647 into main Mar 31, 2026
20 checks passed
@nathanjmcdougall nathanjmcdougall deleted the copilot/dont-run-test-bad-commit-online branch March 31, 2026 23:58
nathanjmcdougall added a commit that referenced this pull request Apr 1, 2026
…ing ones (#1659)

* Initial plan

* Configure agents to fix all static check failures including pre-existing ones

- Add "Fix all failures, including pre-existing ones" section to
  usethis-qa-static-checks SKILL.md (v1.6→v1.7)
- Strengthen usethis-qa-static-checks guidance in AGENTS.md
- Update usethis-python-code-modify SKILL.md (v1.3→v1.4) to reinforce
  the requirement

Resolves #1676

Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/f86eed16-423e-469c-87e9-6f84dbfe3755

Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>
Co-authored-by: Nathan McDougall <nathan.j.mcdougall@gmail.com>
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.

Don't run the test_bad_commit test online on CI

2 participants