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

Add args to the task prefix #4250

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

borissmidt
Copy link

When running the same task with different arguments they are all printed the same.
So make it clear which task ran i propose to add the args to the prefix.

I had a stab at implementing the task dependency args but my rust skills/ knowledge of the project is a bit too limited for that.

@jdx
Copy link
Owner

jdx commented Jan 28, 2025

I do not think this a good idea. There may be many args which will ruin the formatting.

@borissmidt
Copy link
Author

But then how would you keep the tasks apart.
As an alternative could the prefix be truncated at 20 characters or something?

@jdx
Copy link
Owner

jdx commented Jan 28, 2025

You could make the display_name method dispatch to a template

@borissmidt
Copy link
Author

I'll have a look.

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

Successfully merging this pull request may close these issues.

2 participants