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
Show warning when Cake is out of date #2260
Comments
I've been considering this. One approach could to be to write a small class (e.g.
Of course, you might prefer the remote version check to be encapsulated in its own class, taking in the Thoughts? |
@sgrassie At first glance seems like this needs to be its own separate feature that you start asynchronously as early as possible, without waiting/blocking the main thread, then after the build finished, check if the async call finished and make a decision:
|
Hey, is this still open for contribution? |
@secretMeedo Yes. |
@augustoproiete Thank you for the quick response, I'll have a look this weekend and let you know if I can do it. |
When you pin the Cake version which is a recommended practice, there's nothing to remind you when it is out of date. I've seen this result in folks working within the constraints of old versions of Cake without realizing there is new goodness available.
If there was a warning shown at the end of each script execution, it would provide awareness to folks who aren't thinking about loading up the Cake website to check manually. Cake scripts in active projects would be less likely to stay behind at very old versions of Cake.
Could a warning be too strong? Information might be a better fit except that it might fail in its purpose of drawing attention during a normal script execution. On the other hand, there's occasionally a CVE associated with a new update.
Since the update check isn't mission-critical, it could be started asynchronously before compiling the script. If there is no network connectivity, the check can time out silently or be canceled silently when the script execution is finished.
There should be a configuration setting for folks to opt out.
The text was updated successfully, but these errors were encountered: