Skip to content

Conversation

@nathanjmcdougall
Copy link
Collaborator

No description provided.

@nathanjmcdougall nathanjmcdougall linked an issue Oct 7, 2025 that may be closed by this pull request
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR makes Bitbucket CI configuration backend agnostic by adding support for the "none" backend alongside the existing UV backend. This allows users to generate Bitbucket pipelines configuration without requiring UV as a dependency manager.

  • Adds "none" backend support to all tool implementations for generating Bitbucket CI steps
  • Creates backend-agnostic functions for determining supported Python versions
  • Updates test coverage to include both UV and none backend scenarios

Reviewed Changes

Copilot reviewed 23 out of 25 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
tests/usethis/_ui/interface/test_interface_ci.py Adds test for maximal config with none backend and renames existing test for clarity
tests/usethis/_ui/interface/maximal_bitbucket_pipelines_none_backend.yml New test fixture file for none backend pipeline configuration
tests/usethis/_integrations/ci/bitbucket/test_steps.py Adds comprehensive tests for script item canonical ordering
tests/usethis/_integrations/ci/bitbucket/test_anchor.py New test file for anchor name extraction functionality
tests/usethis/_integrations/backend/uv/test_python.py Updates imports to use new backend-agnostic function
tests/usethis/_core/test_core_tool.py Adds tests for Bitbucket integration with none backend
tests/usethis/_core/test_core_ci.py Adds test for CI configuration with none backend
tests/docs/test_readme.py Updates codecov token
src/usethis/_ui/interface/tool.py Adds backend parameter to mkdocs function
src/usethis/_tool/impl/*.py Updates all tool implementations to support both uv and none backends
src/usethis/_integrations/python/version.py Adds convenience function for getting Python major version
src/usethis/_integrations/environ/python.py New module for backend-agnostic Python version handling
src/usethis/_integrations/ci/bitbucket/steps.py Adds canonical ordering for script items and none backend support
src/usethis/_integrations/ci/bitbucket/anchor.py Adds ensure-venv script item and anchor extraction function
src/usethis/_core/tool.py Updates requirements.txt generation logic for none backend
pyproject.toml Adds environ layer to import linter configuration

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@codecov
Copy link

codecov bot commented Oct 7, 2025

Codecov Report

❌ Patch coverage is 97.58065% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/usethis/_tool/impl/pre_commit.py 60.00% 2 Missing ⚠️
src/usethis/_test.py 88.88% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@nathanjmcdougall nathanjmcdougall marked this pull request as ready for review October 10, 2025 21:26
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

Copilot reviewed 24 out of 26 changed files in this pull request and generated 2 comments.


Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@codspeed-hq
Copy link

codspeed-hq bot commented Oct 10, 2025

CodSpeed Performance Report

Merging #953 will not alter performance

Comparing 909-make-bitbucket-ci-config-backend-agnostic (9d0dfad) with main (9053086)

Summary

✅ 2 untouched

@nathanjmcdougall nathanjmcdougall merged commit 674c339 into main Oct 13, 2025
18 checks passed
@nathanjmcdougall nathanjmcdougall deleted the 909-make-bitbucket-ci-config-backend-agnostic branch October 13, 2025 17:01
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.

Make bitbucket CI config backend agnostic

2 participants