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

fix: list tasks #1520

Merged
merged 1 commit into from Feb 28, 2024
Merged

fix: list tasks #1520

merged 1 commit into from Feb 28, 2024

Conversation

pd93
Copy link
Member

@pd93 pd93 commented Feb 27, 2024

Fixes #1514

This change removes an optimisation in favour of reducing complexity. There is nothing particularly time consuming that we are skipping in e.Setup() and this code has been the source of several recent bugs including #1514 and #1509.

@andreynering andreynering added the type: bug Something not working as intended. label Feb 28, 2024
@andreynering andreynering merged commit 6c21568 into main Feb 28, 2024
13 checks passed
@andreynering andreynering deleted the fix-list-tasks branch February 28, 2024 22:44
andreynering added a commit that referenced this pull request Feb 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug Something not working as intended.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

task --slient --list-all with remote taskfiles stores cache in wrong directory
2 participants