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

Error message on circular references leads to poor developer experience #1616

Closed
ghuntley opened this issue May 26, 2017 · 1 comment · Fixed by #3142
Closed

Error message on circular references leads to poor developer experience #1616

ghuntley opened this issue May 26, 2017 · 1 comment · Fixed by #3142
Milestone

Comments

@ghuntley
Copy link
Contributor

ghuntley commented May 26, 2017

image

Repro

Task("Package")
    .IsDependentOn("PinNuGetDependencies")
    .Does (() =>
{
});

Task("PinNuGetDependencies")
    .IsDependentOn("Package")
    .Does (() =>
{
});

Recommend changing the error message to be more developer friendly such as "$task cannot recursively depend on $task".

@augustoproiete augustoproiete added this to the v1.x Next milestone Feb 4, 2021
@augustoproiete augustoproiete modified the milestones: v1.x Next, v1.x Next Candidate Feb 10, 2021
augustoproiete added a commit that referenced this issue Oct 14, 2021
@augustoproiete augustoproiete changed the title error message leads to poor developer experience Error message on circular references leads to poor developer experience Nov 8, 2021
@cake-build-bot
Copy link

🎉 This issue has been resolved in version v2.0.0 🎉

The release is available on:

Your GitReleaseManager bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants