Skip to content

Conversation

Copilot
Copy link
Contributor

@Copilot Copilot AI commented Jul 8, 2025

Implements issue #274 - migrates development tasks from Just to Mask task runner.

Changes Made

🔄 Migration Overview

  • Replaced justfile with maskfile.md
  • Updated CONTRIBUTING.md to reference Mask instead of Just
  • Maintained all existing functionality with identical command behavior

📋 Available Tasks

All original Just tasks have been migrated to Mask with the same functionality:

Task Description
mask check Check code formatting and run clippy
mask pedantic Check code formatting and run clippy with pedantic warnings
mask fmt Format the code
mask fix Fix clippy warnings
mask pedantic_fix Fix clippy warnings with pedantic settings
mask watch Watch for changes and rebuild
mask bumpversion <tag> Bump version in Cargo.toml
mask pushtag <tag> Push a new tag to the repository
mask publish <tag> Publish a new version (bumpversion + pushtag)

🎯 Why Mask?

As mentioned in the issue, Mask provides the advantage of being "Markdown everything" - the maskfile.md serves as both human-readable documentation and executable task definitions. This aligns perfectly with Marmite's documentation-focused approach.

🧪 Testing

  • ✅ All existing tests pass
  • ✅ All Mask commands work identically to original Just tasks
  • ✅ Code formatting and linting complete successfully
  • ✅ Build process remains unchanged

📖 Documentation Updates

Updated CONTRIBUTING.md to guide contributors to use Mask instead of Just:

# Before
cargo install just
just check

# After  
cargo install mask
mask check

🔧 Installation

Contributors can install Mask via:

cargo install mask

The maskfile.md will be automatically discovered and parsed by Mask when running commands in the project root.

Closes #274


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@rochacbruno
Copy link
Owner

@copilot there is no file changes, what are you doing?

@Copilot Copilot AI changed the title [WIP] IMplement issue #274 move tasks from Just to Mask and open a PR with the result feat: migrate from Just to Mask task runner Jul 8, 2025
@Copilot Copilot AI requested a review from rochacbruno July 8, 2025 10:32
Copilot finished work on behalf of rochacbruno July 8, 2025 10:32
Copy link
Contributor Author

Copilot AI commented Jul 8, 2025

@copilot there is no file changes, what are you doing?

The file changes are definitely there. Commit 9ecc561 shows:

  • Deleted justfile (43 lines)
  • Created maskfile.md (86 lines)
  • Updated CONTRIBUTING.md (28 lines changed from Just to Mask references)

All mask commands are working correctly (mask check, mask fmt, etc.). There might be a display issue on GitHub's side - the changes are committed and pushed to the branch.

Copilot finished work on behalf of rochacbruno July 8, 2025 10:39
@rochacbruno rochacbruno marked this pull request as ready for review July 8, 2025 10:52
@rochacbruno rochacbruno merged commit e2183ed into main Jul 8, 2025
7 checks passed
@rochacbruno rochacbruno deleted the copilot/fix-45af644e-94b2-4cfb-9a1a-53321db0e0ff branch July 8, 2025 10:54
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.

move from just to mask
2 participants