cake-contrib / Cake.AddinDiscoverer

Tool to aid with discovering information about Cake Addins
MIT License
5 stars 6 forks source link

Remove Cake Watch Advance #201

Closed TomerPacific closed 3 years ago

TomerPacific commented 3 years ago

Related to this issue from the Cake-Build website repository.

Following the specifications, Cake.Watch_Advance was added to the exclusion list.

This is the first step out of two to remove Cake.Watch_Advance from the website component.

@pascalberger

gep13 commented 3 years ago

@Jericho regarding the build failure, should we exclude this task from PR builds?

TomerPacific commented 3 years ago

Awesome. Thanks @pascalberger. Where do we go from here regarding the original issue?

Jericho commented 3 years ago

@Jericho regarding the build failure, should we exclude this task from PR builds?

Yes indeed. I'll improve the build script to avoid running the discoverer when a PR is submitted.

Jericho commented 3 years ago

Looks good to me!

Thank you for your contribution.