-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Cleanup and windows #2
Conversation
And should (?) be redundant anyway?
This PR was squashed before being merged into the main branch. Discussion ---------- Cleanup and windows Hi! 3 things: 1) Remove the different test suites from being run independently. I don't see a lot of value to this and it meant that I was accidentally running NO tests for TailwindBundle, since I didn't have the testsuites config. So I'd like to simplify. 2) Removed what I believe is an unused `phpunit` input 3) Make `runs-on` dynamic so the job could be re-used for windows. Here is a PR using this new setup: SymfonyCasts/tailwind-bundle#4 Commits ------- 41fe3e7 Cleanup and windows
This PR was squashed before being merged into the main branch. Discussion ---------- Adding windows to ci & fixing Windows binary bug Depends on SymfonyCasts/.github#2 * [x] Merge SymfonyCasts/.github#2 and change the CI config back to the `main` branch Fixes #2 Commits ------- 3d7aa04 Adding windows to ci & fixing Windows binary bug
@weaverryan cool, thanks!
I just ported those test suite calls from the repos where it was used historically... but agree, without test suites it should be more simple and robust 👍 I suppose we should rebuild every repo that uses this config to make sure they are green too? UPD: Rebuild just in case, seems everything is good :) |
This PR was squashed before being merged into the master branch. Discussion ---------- Rebuild CI and fix CS After some changes upstream in GitHub CI config: SymfonyCasts/.github#2 Commits ------- 8da63f1 Rebuild CI and fix CS
This PR was squashed before being merged into the main branch. Discussion ---------- Rebuild CI and fix CS After some changes upstream in GitHub CI config: SymfonyCasts/.github#2 Commits ------- 16a7ae4 Rebuild CI and fix CS
Hi!
3 things:
Remove the different test suites from being run independently. I don't see a lot of value to this and it meant that I was accidentally running NO tests for TailwindBundle, since I didn't have the testsuites config. So I'd like to simplify.
Removed what I believe is an unused
phpunit
inputMake
runs-on
dynamic so the job could be re-used for windows.Here is a PR using this new setup: SymfonyCasts/tailwind-bundle#4