Skip to content

Develop#2

Merged
ptr727 merged 9 commits into
mainfrom
develop
Jun 14, 2025
Merged

Develop#2
ptr727 merged 9 commits into
mainfrom
develop

Conversation

@ptr727

@ptr727 ptr727 commented Jun 14, 2025

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings June 14, 2025 21:14

This comment was marked as outdated.

@ptr727 ptr727 requested a review from Copilot June 14, 2025 21:17

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 integrates CSharpier and Husky.Net formatting tools, updates VSCode build/format tasks, adds Husky pre-commit hooks, and revises CI workflow to use the dotnet tool manifest.

  • Added CSharpier and Husky.Net links in README
  • Updated .vscode/tasks.json with new tasks and debug logging
  • Configured Husky (.husky/task-runner.json & pre-commit) and adjusted GitHub Actions to use dotnet tool restore

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
README.md Added links for CSharpier and Husky.Net
.vscode/tasks.json Renamed labels, added tool-update and Husky-run tasks, enabled debug logs
.husky/task-runner.json Defined CSharpier and dotnet format pre-commit tasks
.husky/pre-commit Hook to invoke Husky task runner
.github/workflows/publish-release.yml Switched to dotnet tool restore and adjusted log levels
.config/dotnet-tools.json Declared csharpier and husky tools in the manifest
Comments suppressed due to low confidence (4)

.vscode/tasks.json:5

  • [nitpick] Restore the uppercase 'NET' in the label to '.NET Build' for consistency with common naming conventions.
"label": ".Net Build",

.vscode/tasks.json:16

  • [nitpick] Use consistent uppercase 'NET' in the task label ('.NET Format') to match other task labels.
"label": ".Net Format",

.github/workflows/publish-release.yml:42

  • The pipeline restores only tools in the manifest; add 'dotnet-format' to .config/dotnet-tools.json or reintroduce its installation so 'dotnet format' is available.
          dotnet tool restore

.vscode/tasks.json:82

  • Remove the trailing comma after the last task in the 'tasks' array; JSON does not allow a comma before the closing bracket.
        },

@ptr727 ptr727 merged commit a87c453 into main Jun 14, 2025
6 checks passed
ptr727 added a commit that referenced this pull request Jun 14, 2025
* First commit

* Do not publish to GitHub

* Update PackegeId

* Add write permissions to actions job

* Include JSON data files in release

* Resolve Copilot review suggestions

* Format code

* Add Husky.Net
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.

2 participants