Skip to content

Conversation

@Sergio0694
Copy link
Member

Closes #1007

This PR updates the code fixer for partial properties to support attribute lists with multiple attributes.

PR Checklist

  • Created a feature/dev branch in your fork (vs. submitting directly from a commit on main)
  • Based off latest main branch of toolkit
  • PR doesn't include merge commits (always rebase on top of our main, if needed)
  • Tested code with current supported SDKs
  • Tests for the changes have been added (for bug fixes / features) (if applicable)
  • Header has been added to all new source files (run build/UpdateHeaders.bat)
  • Contains NO breaking changes
  • Every new API (including internal ones) has full XML docs
  • Code follows all style conventions

@Sergio0694 Sergio0694 added mvvm-toolkit 🧰 Issues/PRs for the MVVM Toolkit analyzer 👓 A new analyzer being implemented or updated labels Nov 27, 2024
@Sergio0694 Sergio0694 changed the title Dev/code fixer multiple attributes Support fixing with attribute lists with multiple attributes Nov 27, 2024
@Sergio0694 Sergio0694 merged commit 3533b97 into main Nov 27, 2024
3 checks passed
@Sergio0694 Sergio0694 deleted the dev/code-fixer-multiple-attributes branch November 27, 2024 03:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

analyzer 👓 A new analyzer being implemented or updated mvvm-toolkit 🧰 Issues/PRs for the MVVM Toolkit

Projects

None yet

Development

Successfully merging this pull request may close these issues.

MVVMTK0042 code fix does not work for fields with additional attributes (e.g. NotifyPropertyChangedFor)

1 participant