Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@inforithmics
Copy link
Contributor

…to slnf

Openin Mapsui.Maui.Slnf I disocvered that Mapsui.Experimental was not in the slnf, then I fixed this for all slnf files

Copy link
Member

@pauldendulk pauldendulk left a comment

Choose a reason for hiding this comment

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

Thanks for taking care of this!

It is easy to to forget about this. Perhaps we should add it to the GHAs. However, the build is currently already taking too long. Perhaps we should move this to a scheduled daily build, so it would run one time a day at most. I would see the build fail notification. We could also move other stuff there. Like building the samples with the nuget reference instead of the project reference.

I wondered if we need this many slnfs. I personally always work with the main one. But then if you are focusing on one UI framework it will speed up development when you have just this one UI framework in the slnf. Also, if you are an app developer who is only checking out the Mapsui source code you don't want to have to install all the other UI frameworks to get it to build. So I just convinced myself it is a good thing to have them. 😄

@pauldendulk pauldendulk merged commit 52fcbc5 into Mapsui:main Oct 20, 2025
5 checks passed
@pauldendulk pauldendulk added the PR type: 🛠️ Maintenance Internal changes that don't affect end users (e.g., code cleanup, build tweaks). label Oct 20, 2025
@pauldendulk pauldendulk changed the title Added missing Mapsui.Experimental and mapsui.Experimental.VectorTils … Add missing Experimental projects to slnfs Oct 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR type: 🛠️ Maintenance Internal changes that don't affect end users (e.g., code cleanup, build tweaks).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants