Skip to content

Conversation

@jodavies
Copy link
Collaborator

@jodavies jodavies commented Oct 7, 2025

No description provided.

@coveralls
Copy link

Coverage Status

coverage: 55.77% (+0.5%) from 55.224%
when pulling 1b5988c on jodavies:test-extra-mincer
into 4e3567c on form-dev:master.

@jodavies
Copy link
Collaborator Author

jodavies commented Oct 7, 2025

@tueda are you happy with the changes in actions.yml? It means we wget https://github.com/form-dev/form-packages/archive/refs/tags/v1.0.0.tar.gz twice, but allows one to enable color and mincer independently in principle (not that we currently need to do this).

@tueda
Copy link
Collaborator

tueda commented Oct 7, 2025

I think this is a reasonable choice for now. In principle, they could be combined, but doing so requires some changes and a decision on whether to use a single package or write more complex code that enables a single download to be reused for the two packages.

@jodavies jodavies merged commit 4910b9a into form-dev:master Oct 7, 2025
84 checks passed
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.

3 participants