Skip to content

Conversation

@marko-stanojevic
Copy link
Contributor

Description

This pull request updates the development container configuration and refines dependency management for the project. The main focus is on improving how the dev container sets up user permissions and installs PowerShell dependencies, as well as updating the format and versions of module dependencies.

Dev container improvements:

  • Added explicit userGid and userUid settings to the common-utils feature in .devcontainer/devcontainer.json for better control over container user permissions.
  • Changed the container setup command from postCreateCommand to onCreateCommand, now using PowerShell's Install-PSResource to install dependencies from requirements.json on container creation, streamlining the setup process.
  • Removed commented-out and unused configuration lines for clarity in .devcontainer/devcontainer.json.

Dependency specification updates:

  • Reformatted the requirements.psd1 file to use a more structured hashtable format for each dependency, and updated the PSScriptAnalyzer version to 1.24.0 for improved linting and analysis.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code refactoring
  • Performance improvement
  • Test improvement

Semantic Versioning

  • +semver: major - Breaking changes (1.0.0 → 2.0.0)
  • +semver: minor - New features (1.0.0 → 1.1.0)
  • +semver: patch - Bug fixes (1.0.0 → 1.0.1)
  • +semver: none - No version change (documentation, tests)

Checklist

Code Quality

  • Code follows PowerShell best practices and approved verbs

Testing

  • All new functions have corresponding .Tests.ps1 files
  • New tests added for new functionality

Documentation

  • Comment-based help is complete and accurate
  • Examples included in help documentation
  • README.md updated (if applicable)

Related Issues

Fixes #issue_number

Additional Notes

@marko-stanojevic marko-stanojevic added the enhancement New feature or request label Jan 7, 2026
@marko-stanojevic marko-stanojevic merged commit 8de14a5 into main Jan 7, 2026
@marko-stanojevic marko-stanojevic deleted the feature/devcontainer_dependencies branch January 7, 2026 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants