Skip to content

Conversation

@bCamba
Copy link
Contributor

@bCamba bCamba commented Dec 18, 2024

For issue #208

}

[MethodImpl(MethodImplOptions.AggressiveInlining)]
public Span<char> AppendSpan(int length)
Copy link
Owner

Choose a reason for hiding this comment

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

Can you add an xmldoc?

@linkdotnet
Copy link
Owner

LGTM.

Can you add an entry in the CHANGELOG.md.
This file is used for making release notes. You can add a new ### Added under UNRELEASED

@sonarqubecloud
Copy link

@bCamba
Copy link
Contributor Author

bCamba commented Dec 18, 2024

Done. Let me know if anything else is needed

@linkdotnet
Copy link
Owner

Done. Let me know if anything else is needed

Perfect! Thanks for your work. I will directly merge it and create a release

@linkdotnet linkdotnet merged commit dece245 into linkdotnet:main Dec 18, 2024
3 checks passed
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