Skip to content

chore: Code formatting and repository maintenance updates#20

Open
ZanattaMichael wants to merge 6 commits into
mainfrom
resync-fork-with-master
Open

chore: Code formatting and repository maintenance updates#20
ZanattaMichael wants to merge 6 commits into
mainfrom
resync-fork-with-master

Conversation

@ZanattaMichael

Copy link
Copy Markdown
Owner

Pull Request (PR) description

This PR includes code formatting improvements and repository maintenance updates:

  1. DevContainer Configuration: Converted indentation from tabs to spaces (4-space indentation) for consistency with JSON formatting standards in .devcontainer/devcontainer.json.

  2. Module Manifest Formatting: Aligned property assignments in source/AzureDevOpsDsc.psd1 for improved readability. Also uncommented FunctionsToExport and CmdletsToExport properties that were previously commented out.

  3. Repository Documentation: Added a maintenance notice to README.md indicating that the repository is mostly unmaintained and directing users to the newer DSCv3 version, along with information about seeking a replacement maintainer.

This Pull Request (PR) fixes the following issues

None

Task list

Test Plan

No testing needed - these are formatting and documentation changes with no functional impact.

https://claude.ai/code/session_01UbbFoPzqDhPfU33zHW8Pwf

johlju and others added 6 commits August 28, 2024 17:22
Added note about the repository's maintenance status and recommendation for archiving.
Brings in 5 upstream commits:
- Update README with maintenance status and recommendations
- Update repo files ready for refactor (dsccommunity#39)
- Fix pinning GitVersion to v5
- Workaround bug in AzDevOpProject resource
- Fix pipeline

Conflict resolutions:
- source/AzureDevOpsDsc.psd1: kept fork's moduleVersion (0.0.2),
  PowerShellVersion (7.0), and full DscResourcesToExport list; adopted
  upstream's explicit FunctionsToExport/CmdletsToExport format
- CHANGELOG.md: merged both sides' Added and Changed entries
- .devcontainer/devcontainer.json: kept upstream's space-indented format
  (functionally identical to fork's tab-indented version)
- tests/Unit/Classes/** (18 UU conflicts): took fork's version — the fork
  deliberately removed the 'using module' preamble and InModuleScope wrapper
  in its test restructure
- 23 DU (modify/delete) test files: kept fork's deletion — these old test
  files were reorganized/replaced in the fork's test restructure

https://claude.ai/code/session_01UbbFoPzqDhPfU33zHW8Pwf
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.

5 participants