Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable PublicApiAnalyzers #2782

Merged
merged 5 commits into from
Oct 7, 2024
Merged

Conversation

keegan-caruso
Copy link
Contributor

@keegan-caruso keegan-caruso commented Aug 14, 2024

Enable PublicApiAnalyzers

See: https://github.com/dotnet/roslyn-analyzers/blob/main/src/PublicApiAnalyzers/PublicApiAnalyzers.Help.md

Include internals for assemblies that are marked with InternalsVisibleTo

@keegan-caruso keegan-caruso requested a review from a team as a code owner August 14, 2024 02:16
@keegan-caruso keegan-caruso added the Internal Indicates issue was opened by the IdentityModel team label Aug 14, 2024
@keegan-caruso
Copy link
Contributor Author

AzDO is passing but not the github action, interesting.

@jennyf19
Copy link
Collaborator

jennyf19 commented Aug 14, 2024

AzDO is passing but not the github action, interesting.

You might try updating the github action from preview 4 to 7 for .NET 9

9.0.100-preview.7.24407.12

@keegan-caruso keegan-caruso force-pushed the kecaruso/add-public-api-analyzer branch from 941bc90 to bb07f84 Compare August 15, 2024 00:02
@pmaytak
Copy link
Contributor

pmaytak commented Aug 15, 2024

As we added this mid release, I had to manually adjust shipped vs unshipped using this as reference

What something like this work - checkout the last 8.0.1 commit, run the tool to generate text files, stash them, and move them over to this branch?

How and when are those files generated, especially the for the unshipped API? Does the tool run automatically somehow or manually update the files?

.editorconfig Show resolved Hide resolved
.editorconfig Show resolved Hide resolved
@keegan-caruso
Copy link
Contributor Author

What something like this work - checkout the last 8.0.1 commit, run the tool to generate text files, stash them, and move them over to this branch?

How and when are those files generated, especially the for the unshipped API? Does the tool run automatically somehow or manually update the files?

These are generated by the fixer with the analyzer. But it doesn't work great for per target items. You can generate it, but then have to manually move it.

I think it should be okay as it. At the next release, we move everything over to released.

Copy link
Collaborator

@jennyf19 jennyf19 left a comment

Choose a reason for hiding this comment

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

:shipit:

@keegan-caruso
Copy link
Contributor Author

As we added this mid release, I had to manually adjust shipped vs unshipped using this as reference

What something like this work - checkout the last 8.0.1 commit, run the tool to generate text files, stash them, and move them over to this branch?

How and when are those files generated, especially the for the unshipped API? Does the tool run automatically somehow or manually update the files?

I ended up going a different route, just start with everything unshipped, on the first release using this, move everything to shipped

@jennyf19
Copy link
Collaborator

@keegan-caruso do you want this in?

@brentschmaltz brentschmaltz merged commit 492a823 into dev Oct 7, 2024
6 checks passed
@westin-m westin-m deleted the kecaruso/add-public-api-analyzer branch October 7, 2024 16:44
@jennyf19 jennyf19 added this to the 8.1.2 milestone Oct 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Internal Indicates issue was opened by the IdentityModel team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants