Skip to content

Conversation

@askpt
Copy link
Member

@askpt askpt commented Sep 19, 2025

This PR

This pull request introduces support for Development Containers (devcontainers) to the repository, making it easier for developers to set up a consistent, containerized development environment. The main changes include the addition of a devcontainer configuration file and documentation updates explaining how to use devcontainers with Visual Studio Code and GitHub Codespaces.

Devcontainer support:

  • Added .devcontainer/devcontainer.json with configuration for the development environment, including the .NET SDK, useful VS Code extensions, and required features.

Documentation improvements:

  • Updated CONTRIBUTING.md to include instructions for using devcontainers, detailing prerequisites and steps for setup with VS Code and GitHub Codespaces.

Related Issues

Fixes #574

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
…iner

Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
@askpt askpt linked an issue Sep 19, 2025 that may be closed by this pull request
@askpt askpt marked this pull request as ready for review September 19, 2025 15:25
@askpt askpt requested a review from a team as a code owner September 19, 2025 15:25
@codecov
Copy link

codecov bot commented Sep 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.04%. Comparing base (fdf2297) to head (b7bbbb8).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #584   +/-   ##
=======================================
  Coverage   90.04%   90.04%           
=======================================
  Files          77       77           
  Lines        2873     2873           
  Branches      323      323           
=======================================
  Hits         2587     2587           
  Misses        226      226           
  Partials       60       60           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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 adds Development Container (devcontainer) support to enable a consistent, containerized development environment for the OpenFeature .NET SDK project.

  • Added devcontainer configuration with .NET SDK 9.0/8.0 and useful VS Code extensions
  • Updated contributing documentation with setup instructions for VS Code and GitHub Codespaces

Reviewed Changes

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

File Description
.devcontainer/devcontainer.json New devcontainer configuration with .NET runtime, development tools, and VS Code extensions
CONTRIBUTING.md Added devcontainer setup instructions and prerequisites

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
Signed-off-by: André Silva <2493377+askpt@users.noreply.github.com>
@askpt askpt added this pull request to the merge queue Sep 20, 2025
Merged via the queue into main with commit ce7baa7 Sep 20, 2025
23 checks passed
@askpt askpt deleted the askpt/574-add-devcontainer branch October 3, 2025 09:13
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.

Add devcontainer configuration to the SDK

4 participants