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

Support Cake 0.38.0 #25

Merged
merged 1 commit into from
Feb 4, 2021
Merged

Support Cake 0.38.0 #25

merged 1 commit into from
Feb 4, 2021

Conversation

bdukes
Copy link
Contributor

@bdukes bdukes commented Jan 27, 2021

TFBuild was renamed to AzurePipelines in Cake 0.38. This PR updates the usages and also updates the naming within the Cake.TFBuild.Module to Cake.AzurePipelines.Module.

I think this is the only thing that would prevent this from working on 1.0.0 also.

Let me know if there are any adjustments I can make for you.

Thanks!

@nils-a
Copy link
Contributor

nils-a commented Feb 3, 2021

@bdukes are you in a position to rebase this onto the current develop?

Copy link
Contributor

@nils-a nils-a left a comment

Choose a reason for hiding this comment

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

One minor change, the rest LGTM.

azure-pipelines.yml Outdated Show resolved Hide resolved
TFBuild was renamed to AzurePipelines
Copy link
Contributor

@nils-a nils-a left a comment

Choose a reason for hiding this comment

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

LGTM

@nils-a nils-a merged commit dba5ec3 into cake-contrib:develop Feb 4, 2021
@nils-a
Copy link
Contributor

nils-a commented Feb 4, 2021

@bdukes your changes have been merged, thanks for your contribution 👍

@bdukes
Copy link
Contributor Author

bdukes commented Feb 5, 2021

@nils-a Thanks! Any chance this change could be published sometime soon?

@nils-a
Copy link
Contributor

nils-a commented Feb 5, 2021

@bdukes I'll try 😄
I want to get the milestone ready, then :shipit:

(Only complex things in there are 28 and 29 - 28 has a PR and is being reviewed, 29 is prepared and only waiting for 28 to be merged.)

@bdukes
Copy link
Contributor Author

bdukes commented Feb 6, 2021

👍 thanks for the update

@nils-a nils-a removed the Feature label Feb 10, 2021
@nils-a
Copy link
Contributor

nils-a commented Feb 10, 2021

@bdukes I released version 1.0.0 which supports Cake 0.38.0 (i.e. this PR) - will update to Cake 1.0.0 (and release a new version of Cake.Buildsystems.Module) soon.

@bdukes
Copy link
Contributor Author

bdukes commented Feb 10, 2021

Thanks @nils-a! I saw the release and am already running a build with it. It appears to be working on our project using Cake 1.0.0, are there features you would expect not to work?

@nils-a
Copy link
Contributor

nils-a commented Feb 10, 2021

@bdukes No. I gave it a shakedown using the rc0003 when I tested your PR (nothing complex) and everything was fine.

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.

2 participants