Matt Johnson-Pint
Matt Johnson-Pint
We *could* improve debug messages for such cases. For example, the start of a session looks like this in the debug output: ``` Debug: Envelope '' sent successfully. Payload: {"sdk":{"name":"sentry.dotnet","version":"3.21.0"},"sent_at":"2022-10-06T18:11:36.785535+00:00"}...
I wonder if installing [the Sentry Cocoa SDK](https://github.com/getsentry/sentry-cocoa) separately in the main application would work in this scenario? I mean, it would work for the app, but would Sentry-Unity find...
Generally looks good. Commented with some minor nits. Didn't test it out though - I'd need more context.
@arichorn - It would seem they forgot to include the name of the offending action into the warning message, so it can be quite cryptic. Re-run a job that shows...
> Can you post your .yml? Sure. https://github.com/getsentry/sentry-dotnet/blob/main/.github/workflows/device-tests.yml > Also, I don't know if we can assume the tools are installed on every flavor of runner. The runner images and...
Confirmed. Thanks for the bug report, we'll fix this asap. Also, note you should probably be using `HH:mm` instead of `hh:mm`. (though not the cause of this issue).
Looks like the problem is [here](https://github.com/moment/moment-timezone/blob/44ce91b03aa528ceb3832d13dbcee041f09c577a/moment-timezone.js#L466): ``` javascript function tz (input) { var args = Array.prototype.slice.call(arguments, 0, -1), name = arguments[arguments.length - 1], zone = getZone(name), out = moment.utc.apply(null, args);...
I'd be fine with distributing `moment-timezone` as one package, with a dependency on another `moment-timezone-data` package. That would at least remove the need to increment the moment-timezone version with each...
What is `process.env.TZ` in this case? If it is `undefined` (or `null`, ``, etc.) it will produce this result.
If you're just looking for the local time zone, you probably don't want to guess that it will be well-formed and in the `TZ` environment variable. That's not a guarantee...