Skip to content

Try SDK - #98

Open
sungam3r wants to merge 7 commits into
masterfrom
test
Open

Try SDK#98
sungam3r wants to merge 7 commits into
masterfrom
test

Conversation

@sungam3r

@sungam3rsungam3r commented Jan 9, 2026

Copy link
Copy Markdown
Member

No description provided.

@sungam3rsungam3r mentioned this pull request Jan 9, 2026
Closed
@CptWesley

CptWesley commented Jan 9, 2026

Copy link
Copy Markdown

The .csproj needs to be added to the solution file.
image

@sungam3r

sungam3r commented Jan 9, 2026

Copy link
Copy Markdown
MemberAuthor

It's worth to note in https://dotnet-project-file-analyzers.github.io/sdk

@sungam3r

sungam3r commented Jan 9, 2026

Copy link
Copy Markdown
MemberAuthor

The .csproj needs to be added to the solution file.

OK. Done but now .globalconfig does not work.

@CptWesley

CptWesley commented Jan 15, 2026

Copy link
Copy Markdown

@Corniel Do you have any idea why the .globalconfig might not be working in this situation?

@Corniel

Corniel commented Jan 16, 2026

Copy link
Copy Markdown

@CptWesley Nope. I've spotted issues like these before. I also tried including (a subset) of config using

 <ItemGroup>
<GlobalAnalyzerConfigFilesInclude="..\..\rules.ini" />
</ItemGroup>

But it had no effect.

@github-actions

github-actionsBot commented Feb 16, 2026

Copy link
Copy Markdown

This pull request was marked as stale since it has not been active for a long time

@sungam3rsungam3r removed the stale label Mar 7, 2026
@github-actions

github-actionsBot commented Apr 7, 2026

Copy link
Copy Markdown

This pull request was marked as stale since it has not been active for a long time

@sungam3rsungam3r mentioned this pull request Apr 16, 2026
Closed
Corniel
Corniel reviewed Apr 17, 2026
View reviewed changes
Comment threadsrc/.net.csproj Outdated
@sungam3r@Corniel
Update src/.net.csproj
7aee44e
Co-authored-by: Corniel Nobel <corniel@gmail.com>
sungam3r
sungam3r commented Apr 22, 2026
View reviewed changes
Comment threadsrc/Directory.Build.props Outdated
@github-actions

github-actionsBot commented May 23, 2026

Copy link
Copy Markdown

This pull request was marked as stale since it has not been active for a long time

@sungam3rsungam3r reopened this Aug 13, 2026
@CornielCorniel mentioned this pull request Aug 15, 2026
Open
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@CornielCornielCorniel left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@sungam3r@CptWesley@Corniel