-
Notifications
You must be signed in to change notification settings - Fork 5
chore: add GitHub actions and remove Package wapproj #13
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
Conversation
During the actual publish when we get release CI going, we probably want to make our build matrix manually specify |
App/Package.appxmanifest
Outdated
|
||
<Identity | ||
Name="925b49fc-4648-4967-b4e6-b5473061ee62" | ||
Publisher="CN=dean" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lmfao oops, I guess it defaults to the current user
Adds fmt, test and build jobs using dotnet.
The fmt job currently doesn't match what Rider does, but I think that will be difficult to pull off without using a different formatting tool which I don't want to investigate for now.
Removes the
Package
project in favor of making theApp
project include the necessary stuff for packaging itself. We weren't getting any benefit from it being in a separate package yet and it was not building correctly in dotnet (only in VS) when they were separated.Closes #7