Skip to content
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

Adding windows to ci & fixing Windows binary bug #4

Closed
wants to merge 9 commits into from

Conversation

weaverryan
Copy link
Contributor

@weaverryan weaverryan commented Jun 28, 2023

Depends on SymfonyCasts/.github#2

Fixes #2

@weaverryan weaverryan changed the title [WIP] Adding windows to ci Adding windows to ci & fixing Windows binary bug Jun 28, 2023
weaverryan added a commit to SymfonyCasts/.github that referenced this pull request Jun 28, 2023
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
weaverryan added a commit that referenced this pull request Jun 28, 2023
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 weaverryan closed this Jun 28, 2023
@weaverryan weaverryan deleted the ci-windows branch June 28, 2023 14:55
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.

Error with tailwind:init on Windows platform
1 participant