Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
deansheather merged 7 commits intomainfromdean/actions
Jan 29, 2025

Conversation

deansheather
Copy link
Member

@deansheatherdeansheather commentedJan 28, 2025
edited
Loading

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 thePackage 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

@deansheatherdeansheather changed the titlechore: add GitHub actionschore: add GitHub actions and remove Package wapprojJan 28, 2025
@deansheatherdeansheather marked this pull request as ready for reviewJanuary 28, 2025 05:33
@deansheather
Copy link
MemberAuthor

During the actual publish when we get release CI going, we probably want to make our build matrix manually specifyRuntimeIdentifier andPublishReadyToRun on the publish command.

dotnet/sdk#39141


<Identity
Name="925b49fc-4648-4967-b4e6-b5473061ee62"
Publisher="CN=dean"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Added via Giphy

Copy link
MemberAuthor

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

@deansheatherdeansheather merged commit7f716c8 intomainJan 29, 2025
3 checks passed
@deansheatherdeansheather deleted the dean/actions branchJanuary 29, 2025 07:44
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@spikecurtisspikecurtisspikecurtis approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add GitHub Actions automation
2 participants
@deansheather@spikecurtis

[8]ページ先頭

©2009-2025 Movatter.jp