Skip to content

CI を統合して .NET 10 に更新 - #12

Merged
esnya merged 4 commits into
masterfrom
codex/modernize-ci-and-net10
Aug 28, 2026
Merged

esnya merged 4 commits into
masterfrom
codex/modernize-ci-and-net10

Conversation

@esnya

@esnya esnya commented Apr 21, 2026

Copy link
Copy Markdown
Owner

概要

  • 単一の ci.yml に CI/release workflow を統合
  • .NET 10 ビルドへ更新
  • analyzer で露出した config manager の問題を修正

ローカル検証

  • dotnet restore .\ViveStreamingFaceTrackingForResonite.sln -p:ResonitePath="C:\Program Files (x86)\Steam\steamapps\common\Resonite\"
  • dotnet build .\ViveStreamingFaceTrackingForResonite.sln -c Release --no-restore -p:ResonitePath="C:\Program Files (x86)\Steam\steamapps\common\Resonite\"

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 4a6a1ec847

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/ci.yml Outdated

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: a5655efc2c

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/ci.yml
Comment on lines +3 to +4
permissions:
contents: write

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Restrict write permission to the release job

Limit contents: write to a separate tag-only release job rather than granting it to the entire build job. For same-repository pull requests, this workflow checks out and executes the PR's MSBuild/workflow code with a write-capable GITHUB_TOKEN, so a contributor or compromised build step can use ${{ github.token }} to alter tags, releases, or repository contents even though only the final release command needs write access.

Useful? React with 👍 / 👎.

Comment thread Directory.Build.props
</Reference>
</ItemGroup>
<ItemGroup Condition="'$(FrooxEnginePresent)'!='true'">
<PackageReference Include="Resonite.GameLibs" />

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Keep the locked dependency graph independent of local files

Avoid conditionally removing this package based on whether a local Resonite installation contains FrooxEngine.dll. The committed lock file records Resonite.GameLibs as a direct dependency, but on the normal development path where Resonite is installed this condition evaluates false; dotnet restore --locked-mode then rejects the mismatched graph, while an ordinary restore rewrites the tracked lock file. This makes the newly enabled package lock nonportable between CI's fallback environment and developers' installed-game environment.

Useful? React with 👍 / 👎.

@esnya
esnya merged commit 8b9a6f4 into master Aug 28, 2026
1 check passed
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.

1 participant