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

Add chromeapp input variable #358

Merged
merged 10 commits into from
Dec 1, 2024

Conversation

MickaelFontes
Copy link
Contributor

@MickaelFontes MickaelFontes commented Oct 30, 2024

@nanasess
Copy link
Owner

nanasess commented Nov 1, 2024

Thanks for your contribution!
I will merge when I have Windows support as well.💪

@MickaelFontes
Copy link
Contributor Author

Hey @nanasess my pleasure! I also noticed some random issues on the ci with the MacOS14 runners (maybe xvfb did not have enough time to start properly?)
Ok noted for the merge, you want to add it yourself for the PowerShell/Windows part?

@nanasess
Copy link
Owner

@MickaelFontes
Sorry for the delay in getting back to you due to a cold.
@MickaelFontes san, can you also handle PowerShell/Windows part?

@MickaelFontes
Copy link
Contributor Author

Hi @nanasess no problem!
I added the Windows part, it should be good now
(your review is welcome in case I missed something)

@nanasess
Copy link
Owner

nanasess commented Dec 1, 2024

LGTM

@nanasess nanasess merged commit 5418e30 into nanasess:master Dec 1, 2024
40 checks passed
@nanasess
Copy link
Owner

nanasess commented Dec 3, 2024

@MickaelFontes
Thanks for your contribution! New version has been released.
https://github.com/nanasess/setup-chromedriver/releases/tag/v2.3.0

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.

Make CHROMEAPP an input variable that can be declared from the GitHub action workflow
2 participants