Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Problems with InternalsVisibleTo #441

Open
Arlodotexe opened this issue Jun 27, 2024 · 1 comment · May be fixed by #446
Open

Problems with InternalsVisibleTo #441

Arlodotexe opened this issue Jun 27, 2024 · 1 comment · May be fixed by #446

Comments

@Arlodotexe
Copy link
Member

Worth noting our use of InternalsVisibleTo are using or being used by same set of components that keep appearing in these issues-- Behaviors, Animations, Media and Extensions.

If our use of InternalsVisibleTo is indeed related to our AoT issues on Wasdk as well as what we've seen here regarding transient dependencies, it might be highlighting a deeper issue with how we've set up these libraries.

We might want to go through and see if we can remove the need for these InternalsVisibleTo, it looks like at least a few are just to avoid helper duplication. If it fixes any of the issues we've listed throughout this ticket, we've narrowed the scope of our search down considerably.

Originally posted by @Arlodotexe in #169 (comment)

@Arlodotexe Arlodotexe moved this to 🔖 Ready in Toolkit 8.x Jun 27, 2024
@Arlodotexe Arlodotexe moved this from 🔖 Ready to 🏗 In progress in Toolkit 8.x Jul 1, 2024
@Arlodotexe
Copy link
Member Author

Arlodotexe commented Jul 2, 2024

Adding additional details highlighted in #446:

We're testing these changes for three reasons:

The third reason is dismissed, CommunityToolkit/Tooling-Windows-Submodule#195 (comment) helped us verify that it was unrelated, and the fix offered by Mano and yourself @Sergio0694 double confirms that.

The first two are still a possible concern. If we can verify that the use of InternalsVisibleTo here isn't uncovering some platform bug causing #169, we can dismiss that as well and close this PR without merging.

@Arlodotexe Arlodotexe moved this from 🏗 In progress to 📋 Backlog in Toolkit 8.x Jul 8, 2024
@Arlodotexe Arlodotexe moved this from 📋 Backlog to 🏗 In progress in Toolkit 8.x Jul 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 📋 Backlog
Development

Successfully merging a pull request may close this issue.

1 participant