@wakco commented on GitHub:
Added since first posting, checking the token is valid and has the necessary permissions, and making sure the rate limit is only checked when using the api
@wakco commented on GitHub:
While this does help with computers "... behind a firewall presenting one ... NAT'd IP address...", no I do not believe Installomator should be specifically trying to…
@BigMacAdmin commented on GitHub:
Can you provide references to where this was previously discussed/rejected so that the team can review?
Looking at your fork, I see what appears to be more…
@wakco commented on GitHub:
Yes, the GITHUBAPI variable needs to be before the labels are processed, and technically the previous change away from processing them before actually broke both the…
@abstertee commented on GitHub:
Is this still an issue? This is working fine for me with the 10.8 version.
@wakco commented on GitHub:
I've updated the processCommandLineArguments function in functions.sh file, restoring the processing of unrecognised command line options, while getting stricter…
@wakco commented on GitHub:
Here,
Pull request https://github.com/Installomator/Installomator/pull/22 links to issue https://github.com/Installomator/Installomator/issues/23 also mentions this…
@wakco commented on GitHub:
While I am sure it was rejected, not finding any evidence of that, so I have updated the "rejected" mentions to closed since the only real references I have found so…
@acodega commented on GitHub:
Their latest release only includes a Windows version. Should this label kept anymore? Last update was in 2023 and according to their README it seems like active…
@ReivenShadowmoon commented on GitHub:
Closing as @jvaglivelo has this handled on his submission #2181.
@jeanniekhouri commented on GitHub:
Their latest release only includes a Windows version. Should this label kept anymore? Last update was in 2023 and according to their README it seems like…
@acodega commented on GitHub:
That link is for the project as a whole, they have a desktop repo that's more relevant.
@acodega commented on GitHub:
There's several issues and PRs already discussing this.
The PR fix should be merged soon. Closing this for now as a dup.