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

Fix #133

Merged
merged 30 commits into from
Oct 25, 2023
Merged

Fix #133

merged 30 commits into from
Oct 25, 2023

Conversation

ckvsoft
Copy link
Contributor

@ckvsoft ckvsoft commented Oct 25, 2023

Finally everything is GREEN.
The macos stuff almost cracked me

BUT: with the pull request there is an error during run that it cannot get the controller.sh from the "pull" branch
Let's see how we can fix that too

ckvsoft and others added 30 commits October 25, 2023 11:22
duplicate function declaration removed
Code alignment fixes
add ACTOR
Fix BRANCH and ACTOR
fix BRANCH name
BRANCH name
Fix CONFIG-VERSION
Fix: if bash version is less than 4, only CONFIG-VERSION is checked
Fix missing CONFIG-VERSION
Fix: I hope it finally works on the Mac too ;)
that still drives me crazy. Investing so much time in something that doesn't interest me at all. Who runs a script like that on a Mac? don't see any point in it
fix trailing spaces
fix trailing spaces
@christian1980nrw christian1980nrw merged commit 028f620 into christian1980nrw:dev Oct 25, 2023
5 checks passed
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.

3 participants