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

DO NOT MERGE! Github Actions build + annotations test #3371

Closed
wants to merge 34 commits into from

Conversation

AlanRynne
Copy link
Member

This is just a PR to test a quick Github actions build pipeline I created last night.

we shouldn't actually merge this, it's just a POC. I need the PR to verify PR annotations are coming through and check if the result provides better DX for the team.

- name: Restore projects
run: dotnet restore DUI3-DX.slnf
- name: Build
run: msbuild DUI3-DX.slnf /p:Configuration=Release /p:IsDesktopBuild=false
Copy link
Member

Choose a reason for hiding this comment

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

not sure if this helps but using bullseye and C# instead of msbuild is what I do: https://github.com/adamralph/bullseye

Copy link
Member Author

Choose a reason for hiding this comment

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

Nice! I've been eyeing some build systems for a while but didn't know about this one. Adding it to my "must look into" list!

Copy link
Member

Choose a reason for hiding this comment

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

I like that the build is the same language as the project and not XML.

I used to use (and contribute) to Cake but thought it was overblown. Found bullseye and used it ever since dotnet got simple and fast

@AlanRynne
Copy link
Member Author

Closing to reduce noise, @adamhathcock has his branch ongoing and we'll see what we do with this once we reach somewhere meaningful.

@AlanRynne AlanRynne closed this May 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants