Feat: devcontainer pytest integration #110
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Author
@microsoft-github-policy-service agree [company="microsoft"] |
junkataoka
force-pushed
the
feature/devcontainer-pytest-integration
branch
from
44faaca to
ec09cba
Compare
Author
|
@microsoft-github-policy-service agree |
junkataoka
force-pushed
the
feature/devcontainer-pytest-integration
branch
5 times, most recently
from
2523a23 to
0ebed45
Compare
junkataoka
changed the title
junkataoka
changed the title
junkataoka
force-pushed
the
feature/devcontainer-pytest-integration
branch
from
d73bb8e to
82fb7b9
Compare
junkataoka
force-pushed
the
feature/devcontainer-pytest-integration
branch
from
fe5a82e to
78f787f
Compare
junkataoka
force-pushed
the
feature/devcontainer-pytest-integration
branch
3 times, most recently
from
f22f8ef to
d94a2d6
Compare
- Add GitHub Actions workflow using official devcontainers/ci@v0.3 action - Add Azure DevOps pipeline with custom templates for macOS compatibility - Configure pytest with coverage reporting and JUnit XML output - Handle workspace permission issues using /tmp with sudo - Support CPU, GPU, and notebooks project configurations - Add coverage merging and test result publishing for both platforms - Skip pre-commit hook installation in CI when .git not available - Add sync-to-azdo workflow for pipeline synchronization - Update documentation with CI/CD setup instructions GitHub Actions uses the official action which works well on Linux. Azure DevOps uses custom templates to handle macOS-specific Docker issues (path translation and working directory mount namespace errors).
junkataoka
force-pushed
the
feature/devcontainer-pytest-integration
branch
from
672b9c6 to
164e6b9
Compare
junkataoka
force-pushed
the
feature/devcontainer-pytest-integration
branch
from
a279838 to
5f6316e
Compare
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 15 out of 16 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
junkataoka
force-pushed
the
feature/devcontainer-pytest-integration
branch
from
0e665e8 to
5f6316e
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request introduces a major refactor and modernization of the CI pipeline infrastructure for both Azure DevOps and GitHub Actions. The main improvements are the switch to modular, template-driven pipeline definitions, enhanced support for devcontainer-based testing, and improved code coverage reporting. The changes streamline CI setup, make it easier to add new projects, and improve compatibility across platforms (Linux and macOS). Documentation and configuration files have also been updated for clarity and consistency.
Azure DevOps Pipeline Modernization
.azuredevops/ado-ci-pipeline-ms-hosted.ymland.azuredevops/ado-ci-pipeline-self-hosted.ymlare refactored to use a two-stage pipeline (Lint and Test), with jobs and steps defined via new templates for devcontainer testing and coverage publishing. This replaces legacy scripts and direct task definitions, making the pipeline more maintainable and extensible. [1] [2] [3]Template-Driven CI Architecture
.azuredevops/templates/, includingtest-devcontainer-job.ymlfor per-project devcontainer testing,run-devcontainer.ymlfor platform-aware container execution,setup-devcontainer.ymlfor environment setup,publish-test-results.ymlfor publishing test results and coverage, andmerge-coverage.ymlfor merging and publishing code coverage reports. [1] [2] [3] [4] [5]GitHub Actions Pipeline Improvements
.github/workflows/ci.yamlis reworked to use matrix jobs for devcontainer-based testing, artifact upload for results and coverage, and a dedicated lint job. A new job aggregates and publishes combined test results and coverage reports, improving visibility and maintainability. [1] [2]Documentation and Configuration Updates
ci-tests.sh) are removed fromREADME.mdand.env.exampleis updated for clarity, reflecting the new devcontainer and pipeline workflow. [1] [2] [3]Platform Compatibility and Usability Enhancements
Does this introduce a breaking change?
Author pre-publish checklist
Pull Request Type
What kind of change does this Pull Request introduce?