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

Enhance CI integration of Cake #4439

Merged
merged 3 commits into from
Feb 10, 2021
Merged

Conversation

bdukes
Copy link
Contributor

@bdukes bdukes commented Jan 26, 2021

This PR integrates the Cake tool Cake.BuildSystems.Module:

This module will introduce a number of features for running in hosted CI build environments to tightly integrate with the host environment/tools. These modules require no changes to build scripts and instead rely on your Cake script's standard aliases and lifecycle events to integrate your script into the environment.

For Azure DevOps it will provide the following

  • Individual timeline records for each task
  • Percentage reporting on build progress
  • Integrates Warning and Error logging aliases with the Build Issues summary
  • Includes a Cake Build Summary widget on the build summary page

@bdukes
Copy link
Contributor Author

bdukes commented Jan 26, 2021

This doesn't seem to have had an effect, I have an open issue on the tool repo to investigate if I'm doing something wrong.

@bdukes
Copy link
Contributor Author

bdukes commented Jan 27, 2021

I (temporarily) directly added a custom version of the NuGet package which works for Cake 1.0.0, you can see the extra build info at https://dev.azure.com/dotnet/DNN/_build/results?buildId=44680&view=ms.vss-build-web.run-extensions-tab. I don't see that there's enough info in there to worry about integrating this Cake module.

My next attempt will be looking at integrating https://cakeissues.net/, which can read MSBuild logs and report issues to the build, as well as to the PR as a comment (if we want that).

@bdukes
Copy link
Contributor Author

bdukes commented Feb 5, 2021

This PR is in a testing state where I'm using manually build DLLs instead of a NuGet package. My change cake-contrib/Cake.BuildSystems.Module#25 was merged, but waiting for a new version to be published first.

@valadas
Copy link
Contributor

valadas commented Feb 9, 2021

@bdukes is this good to go now or still in testing state?

@valadas valadas modified the milestones: 9.9.0, 9.9.1 Feb 9, 2021
@bdukes
Copy link
Contributor Author

bdukes commented Feb 10, 2021

This is available to merge now (no rush)

@valadas valadas merged commit df51959 into dnnsoftware:develop Feb 10, 2021
@bdukes bdukes deleted the cake-issues branch February 12, 2021 21:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants