Skip to content

Fix XML Comment Warnings#137

Merged
milkshakeuk merged 3 commits intonHapiNET:masterfrom
AMCN41R:128_XmlCommentRefactor
Feb 1, 2021
Merged

Fix XML Comment Warnings#137
milkshakeuk merged 3 commits intonHapiNET:masterfrom
AMCN41R:128_XmlCommentRefactor

Conversation

@AMCN41R
Copy link
Copy Markdown
Collaborator

@AMCN41R AMCN41R commented Jan 30, 2021

Fixes XML comment warnings in NHapi.Base and NHapi.SourceGeneration projects.
issue: #128

@AMCN41R AMCN41R mentioned this pull request Jan 30, 2021
@AMCN41R AMCN41R changed the title Fix XML Comment Warnings #128 Fix XML Comment Warnings Jan 30, 2021
@AMCN41R AMCN41R changed the title #128 Fix XML Comment Warnings Fix XML Comment Warnings Jan 30, 2021
Copy link
Copy Markdown
Member

@milkshakeuk milkshakeuk left a comment

Choose a reason for hiding this comment

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

I think there a couple here which might need amending, the others are more related to incorrect code examples which this ticket is not trying to resolve.

Comment thread src/NHapi.Base/Model/ExtraComponents.cs Outdated
Comment thread src/NHapi.Base/Model/GenericPrimitive.cs
Comment thread src/NHapi.Base/Model/ISegment.cs Outdated
Comment thread src/NHapi.Base/Model/Primitive/DT.cs
Comment thread src/NHapi.Base/Model/Primitive/TSComponentOne.cs
Comment thread src/NHapi.Base/Model/Varies.cs Outdated
Comment thread src/NHapi.Base/Model/Varies.cs Outdated
Comment thread src/NHapi.Base/Parser/DefaultModelClassFactory.cs
Comment thread src/NHapi.Base/Parser/ParserBase.cs
Comment thread src/NHapi.Base/Util/Terser.cs
@AMCN41R
Copy link
Copy Markdown
Collaborator Author

AMCN41R commented Jan 31, 2021

@milkshakeuk - thanks for reviewing. I think i've covered those off.

Comment thread src/NHapi.Base/Model/ISegment.cs
Copy link
Copy Markdown
Member

@milkshakeuk milkshakeuk left a comment

Choose a reason for hiding this comment

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

@AMCN41R Thanks for this.

@AMCN41R AMCN41R force-pushed the 128_XmlCommentRefactor branch from ddc7ae7 to dd566dc Compare February 1, 2021 09:14
@AMCN41R AMCN41R force-pushed the 128_XmlCommentRefactor branch from dd566dc to 0f01237 Compare February 1, 2021 09:20
@milkshakeuk milkshakeuk merged commit 0994b3d into nHapiNET:master Feb 1, 2021
@AMCN41R AMCN41R deleted the 128_XmlCommentRefactor branch February 1, 2021 09:34
@milkshakeuk milkshakeuk linked an issue Feb 1, 2021 that may be closed by this pull request
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.

Fix XML Build warnings

2 participants