Skip to content

fixes #242: add missing attribute targets#243

Merged
nojaf merged 2 commits intofsprojects:masterfrom
dlidstrom:master
May 17, 2024
Merged

fixes #242: add missing attribute targets#243
nojaf merged 2 commits intofsprojects:masterfrom
dlidstrom:master

Conversation

@dlidstrom
Copy link
Contributor

I've added AttributeTargets.Method to all attributes which have AttributeTargets.Property. I also added to obsolete attributes thinking it is necessary to avoid forcing removal of those attributes.

@nojaf
Copy link
Collaborator

nojaf commented May 17, 2024

Thanks Daniel! Could you add a new entry in the release notes (https://raw.githubusercontent.com/fsprojects/Argu/master/RELEASE_NOTES.md).

You can use 6.2.4, if that is present there should be a new package after merging this PR.

@dlidstrom
Copy link
Contributor Author

@nojaf I've updated the release notes. Is it ok?

Copy link
Collaborator

@nojaf nojaf left a comment

Choose a reason for hiding this comment

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

Wonderful, thank you!

@nojaf
Copy link
Collaborator

nojaf commented May 24, 2024

Sorry, I assumed merging to master would publish to NuGet.
That did happen and I triggered it manually, https://www.nuget.org/packages/Argu/6.2.4

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