Skip to content

chore(WD-33708): Add Taskfile on assets manager#312

Open
muhammad-ali-pk wants to merge 4 commits intomainfrom
chore/taskfile
Open

chore(WD-33708): Add Taskfile on assets manager#312
muhammad-ali-pk wants to merge 4 commits intomainfrom
chore/taskfile

Conversation

@muhammad-ali-pk
Copy link
Contributor

@muhammad-ali-pk muhammad-ali-pk commented Feb 12, 2026

Done

  • Added taskfile support to run project

QA

  • Checkout this pull request
  • Make sure you have the following latest versions installed and available on your system
  • Run the project using task
  • Verify the project runs as expected, and accessible on localhost:8017
  • Verify all the github actions run as expected
    • Please note: The lint-python check is failing, which is not reproducible locally. As the files mentioned in this check are irrelevant to the PR, it can be fixed later in a maintenance task.

Issue / Card

Fixes WD-33708

@muhammad-ali-pk muhammad-ali-pk force-pushed the chore/taskfile branch 5 times, most recently from 74d6e7c to 49fcf03 Compare February 12, 2026 08:59
@muhammad-ali-pk muhammad-ali-pk changed the title Chore/taskfile chore(WD-33708): Add Taskfile on assets manager Feb 12, 2026
@muhammad-ali-pk muhammad-ali-pk requested a review from a team February 12, 2026 11:00
Copy link

@Skazitron Skazitron left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@muhammad-ali-pk LGTM! I'll approve but consider adding linting, formatting, and tests to this. we should probably make it support all of the commands that dotrun currently accepts.


rebuild:
desc: "Rebuild the project. Only run this if you have made changes to the Docker related files (Dockerfile, docker-compose.yaml)."
cmds:

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It might be a good idea to include linting and formatting here

@Onibenjo
Copy link
Contributor

For the lint failing check

Fix it by either:

  • pinning Black either by adding it to requirements.txt like this or
  • explicitly pinning it in the CI step like this.

as indicated here

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants