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 task definitions for duty as alternative command runner #121

Closed
wants to merge 4 commits into from

Conversation

dalito
Copy link
Contributor

@dalito dalito commented Aug 19, 2024

Adds a duties.py tasks file for the duty command runner.

The added commands should match the current makefile. The code for the tasks in duties.py may be made nicer given the power of Python but I first wanted to closely follow the makefile.

Closes #120

@pkalita-lbl
Copy link
Contributor

pkalita-lbl commented Aug 19, 2024

Not to derail this but I've had my eye on Task as a cross-platform alternative to make, but I've never really had a good excuse to use it. I've never seen duty before. It seems interesting, but I'm just mulling over the tradeoffs of tasks defined via code vs configuration.

EDIT: Whoops! I just saw in this issue that you already did consider Task as an option.

@dalito
Copy link
Contributor Author

dalito commented Aug 21, 2024

@pkalita-lbl - Task or just are probably a better choice for someone who does not know/like Python or for developers who want one tool for all their projects (also Python-free ones). The preference for duty here is only driven by the desire to be compatible with the installation process and to cause least challenges for newcomers.

@dalito
Copy link
Contributor Author

dalito commented Dec 13, 2024

The selection of duty was mainly motivated by its ease of installation. Users would just need to have pipx to install all tools needed.

In the meantime just can be installed via pipx as well. This is a nice surprise since the just devs said "not planned" when I asked for it a while ago. Since just is a much wider used tool, I am closing this PR in favor of a future just-based PR.

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.

Alternative command runner to make (tasks/just/duty)
2 participants