This is regarding the changes in #4831 (review)
Our logs are already very long. If we are on AppVeyor and not local, it may be better to add these logs as artifacts and reference them, If local we can just reference the logs.
Here are the docs on how to push an artifact
https://www.appveyor.com/docs/build-worker-api/#push-artifact
You can test if we are in AppVeyor using an environment variable:
https://www.appveyor.com/docs/environment-variables/
This is regarding the changes in #4831 (review)
Our logs are already very long. If we are on AppVeyor and not local, it may be better to add these logs as artifacts and reference them, If local we can just reference the logs.
Here are the docs on how to push an artifact
https://www.appveyor.com/docs/build-worker-api/#push-artifact
You can test if we are in AppVeyor using an environment variable:
https://www.appveyor.com/docs/environment-variables/