Skip to content

Align preamble for generated files with core emitter#93

Merged
andreasohlund merged 8 commits into
mainfrom
emit
May 11, 2026
Merged

Align preamble for generated files with core emitter#93
andreasohlund merged 8 commits into
mainfrom
emit

Conversation

@andreasohlund
Copy link
Copy Markdown
Member

This makes sure that we suppress obsolete warnings, CS1591 and also emit the GeneratedCodeAttribute.

Comment thread src/NServiceBus.AzureFunctions.Analyzer/Utility/SourceWriterExtensions.cs Outdated
writer.WithOpenNamespace(composition.RootNamespace);
writer.WriteLine("public static class NServiceBusFunctionsComposition");
writer.WithGeneratedCodeAttribute();
writer.WriteLine("internal static class NServiceBusFunctionsComposition");
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I made this one internal

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.

On the fence about this one but I also don't have a good counter argument for not making it internal

@andreasohlund andreasohlund merged commit 32d70d9 into main May 11, 2026
6 of 8 checks passed
@andreasohlund andreasohlund deleted the emit branch May 11, 2026 05:24
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