Skip to content

Comments

Fix CI and simplify testing infrastructure#1

Merged
jdx merged 11 commits intomainfrom
fix-ci-and-simplify
Sep 13, 2025
Merged

Fix CI and simplify testing infrastructure#1
jdx merged 11 commits intomainfrom
fix-ci-and-simplify

Conversation

@jdx
Copy link
Member

@jdx jdx commented Sep 13, 2025

Summary

This PR fixes the failing CI tests and simplifies the testing/linting infrastructure as requested.

Changes

  • ✅ Remove unnecessary options from mise-action
  • ✅ Use mise to install lua instead of apt
  • ✅ Remove release workflow
  • ✅ Simplify linting to inline commands in mise.toml
  • ✅ Simplify test script to inline commands in mise.toml
  • ✅ Remove separate script files (scripts/*.sh)

Test Plan

All tasks have been tested locally:

  • mise run lint - Validates Lua syntax
  • mise run test - Tests plugin installation and execution
  • mise run ci - Runs both lint and test

The simplified CI workflow should now pass on GitHub Actions.

🤖 Generated with Claude Code

jdx and others added 11 commits September 13, 2025 08:13
- Remove unnecessary options from mise-action
- Use mise to install lua instead of apt
- Remove release workflow
- Simplify lint and test tasks to inline commands in mise.toml
- Remove separate script files

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Run `mise run ci` directly instead of separate lint/test jobs

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Replace basic Lua syntax check with luacheck
- Add StyLua for consistent code formatting
- Add configuration files for both tools
- Format all Lua files with StyLua
- Fix unused variable warnings from luacheck
- Add format task to mise.toml

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Simpler and faster to use the direct stylua plugin

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Add semver installation step to test task
- Remove semver from tools to avoid conflicts during testing
- Set MISE_USE_VERSIONS=0 globally in mise.toml

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Use MISE_USE_VERSIONS_HOST instead of MISE_USE_VERSIONS

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Move test logic to .mise/tasks/test
- Move lint logic to .mise/tasks/lint
- Simplify mise.toml by removing inline scripts
- Tasks are now more maintainable as separate files

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Use `mise plugin link --force` instead of uninstall/link
- Remove stderr redirect from luarocks install for better debugging

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
More conventional directory name for mise task files

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Test now displays the actual semver output for visibility
while still verifying the version is correct

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Added #MISE description comments to file tasks for better documentation

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@jdx jdx merged commit 9621f14 into main Sep 13, 2025
2 checks passed
@jdx jdx deleted the fix-ci-and-simplify branch September 13, 2025 13:30
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.

1 participant