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

Colors for new agents & Improved configurator.bat #209

Merged
merged 3 commits into from
Jan 9, 2025

Conversation

pintoso
Copy link
Contributor

@pintoso pintoso commented Jan 8, 2025

  • Added colors for harbor, gekko, vyse, iso, clove, tejo

  • Improved configurator.bat to show warns and work properly with the Python version.

@OwOHamper
Copy link
Collaborator

Please use python instead of py, it is a lot more common.

Also what is the idea behind redirecting the output to null?

@pintoso
Copy link
Contributor Author

pintoso commented Jan 9, 2025

Also what is the idea behind redirecting the output to null?

To hide the errors that Windows throws in case the .exe or .py file is missing, making it easier to read the terminal with the defined messages.
I don't see the need for the user to see these Windows errors since there are only two reasons for the error:

  • The .exe is not in the folder / has a different name
  • The main.py is not in the folder / Python is not installed

@OwOHamper
Copy link
Collaborator

Okay agreed, LGTM

@OwOHamper OwOHamper merged commit 2e585c4 into zayKenyon:main Jan 9, 2025
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.

2 participants