search

ME-MarvinE / XCalendar

A plugin for .NET providing an API for representing a calendar along with fully customisable calendar controls for Xamarin Forms and .NET MAUI
MIT License
300 stars 36 forks source link

remove unnecessary tags on csproj at samples #159

Closed migueBarrera closed 1 year ago

migueBarrera commented 1 year ago

We can delete all these lines, having a cleaner and more maintainable file.

migueBarrera commented 1 year ago

Reject the proposal if you don't see it as correct; I can't help but do some 'boyscouting' when reading code.

ME-MarvinE commented 1 year ago

I saw these before and didn't know what these did and I didn't notice any change in behaviour when removing them so I just left them alone.

But if you say we don't need them then that's great.

ME-MarvinE commented 1 year ago

Getting "Something went wrong" when attempting to merge. Will try again later.

migueBarrera commented 1 year ago

Great, thank you very much. It works by naming convention, if you call the .cs the same as the .xaml, you don't have to declare it in the csproj. You can have cleaner csproj.

ME-MarvinE commented 1 year ago

Ahh okay, thank you for explaining.