Skip to content

Conversation

goulinkh
Copy link
Contributor

Done

  • Introduce taskfile
  • Introduce mise.dev to replace docker container mode
  • Update gitignore

QA

  • Check out this feature branch
  • Run the site using the command task
  • View the site locally in your web browser at: http://0.0.0.0:8001/
    • Be sure to test on mobile, tablet and desktop screen sizes

Help

QA steps - Commit guidelines

@webteam-app
Copy link

Copy link

codecov bot commented Sep 15, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 57.09%. Comparing base (9e10a80) to head (03b191c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #15610      +/-   ##
==========================================
+ Coverage   47.86%   57.09%   +9.22%     
==========================================
  Files          36      163     +127     
  Lines        5563     9213    +3650     
  Branches        0     1264    +1264     
==========================================
+ Hits         2663     5260    +2597     
- Misses       2900     3924    +1024     
- Partials        0       29      +29     

see 127 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@samhotep
Copy link
Member

Hey @goulinkh could you please also add instructions to the README.md

@goulinkh goulinkh force-pushed the taskfile-integration branch 3 times, most recently from 98c9ace to 62a7a1d Compare September 15, 2025 13:41
@goulinkh goulinkh force-pushed the taskfile-integration branch from 62a7a1d to 940cbfb Compare September 15, 2025 13:47
- install-yarn
cmds:
- PATH="{{ .VENV_BIN }}:$PATH" yarn build

Copy link
Member

Choose a reason for hiding this comment

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

Sorry, should have mentioned in the first comment:
It might be helpful to also have an option for clean

@samhotep
Copy link
Member

@goulinkh how can we use the mise environment from the cli once it has been set up? For example to run a local script with the created virtual environment?

@goulinkh goulinkh force-pushed the taskfile-integration branch from 940cbfb to 32c5ea0 Compare September 15, 2025 13:52
@goulinkh goulinkh force-pushed the taskfile-integration branch 4 times, most recently from 792d333 to 47167dd Compare September 15, 2025 15:01
@Copilot Copilot AI review requested due to automatic review settings September 15, 2025 17:08
@goulinkh goulinkh force-pushed the taskfile-integration branch from 47167dd to 0757bf9 Compare September 15, 2025 17:08
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces Taskfile to replace dotrun as the primary development tool for the project. It modernizes the development workflow by adding task-based automation and mise.dev for dependency management.

  • Introduces a comprehensive taskfile.yaml with tasks for development, testing, and building
  • Updates documentation to recommend Taskfile over dotrun
  • Adds GitHub Actions workflow for CI/CD using Taskfile

Reviewed Changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated 3 comments.

File Description
taskfile.yaml Comprehensive task configuration with development, testing, and build automation
README.md Updates documentation to recommend Taskfile and deprecate dotrun
.github/workflows/pr-with-task.yaml New CI/CD workflow using Taskfile for automated testing
.env Minor environment variable updates including new SEARCH_API_KEY

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@goulinkh goulinkh force-pushed the taskfile-integration branch from 0757bf9 to 3cce8c4 Compare September 15, 2025 17:16
@goulinkh goulinkh force-pushed the taskfile-integration branch from 3cce8c4 to 03b191c Compare September 15, 2025 17:21
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