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 keegan-caruso force-pushed the kecaruso/add-public-api-analyzer branch from 0c4f8b7 to 38c3f4e Compare September 10, 2024 02:48
@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

@keegan-caruso keegan-caruso force-pushed the kecaruso/add-public-api-analyzer branch from 6556c6e to 6368acd Compare September 10, 2024 15:55
@jennyf19
Copy link
Collaborator

@keegan-caruso do you want this in?

@keegan-caruso keegan-caruso force-pushed the kecaruso/add-public-api-analyzer branch from 59ef7bc to 6d613d1 Compare September 20, 2024 23:00
@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