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

feat: logging in windows environment #450

Merged
merged 2 commits into from Apr 8, 2024

Conversation

SMoraisAnsys
Copy link
Contributor

This PR ensures that a self-hosted runner recent enough, i.e. using windows 10, created without specific setup is able to use the _logging private action. Before it was only working if bash was included in PATH which is not the case for many self hosted runners.

This will allow future actions dedicated to windows could use the _logging action without breaking changes.

@ansys-reviewer-bot
Copy link
Contributor

Thanks for opening a Pull Request. If you want to perform a review write a comment saying:

@ansys-reviewer-bot review

@SMoraisAnsys
Copy link
Contributor Author

This worked with linux and windows github runners and a windows self-hosted runner used by pyaedt toolkits (see https://github.com/ansys/actions/actions/runs/8600801687/)

@SMoraisAnsys SMoraisAnsys added the enhancement New features or code improvements label Apr 8, 2024
@SMoraisAnsys SMoraisAnsys marked this pull request as ready for review April 8, 2024 14:14
@SMoraisAnsys SMoraisAnsys requested a review from a team as a code owner April 8, 2024 14:14
@SMoraisAnsys SMoraisAnsys merged commit f60ce81 into ansys:main Apr 8, 2024
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New features or code improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants