[CI] Use Perl Core module JSON::PP #3188

chalin · 2023-08-16T13:48:37Z

Closes [CI] fix:dict fails due to missing JSON package #3187
Contributes to CI improvements - 2023/08 #3122
Switches to using the core Perl module JSON::PP instead of JSON
I tested this on GitPod (which doesn't have the JSON module installed by default), and it works.
@julianocosta89 - if you still have issues, I'll find another solution.

julianocosta89 · 2023-08-16T14:14:28Z

Thanks @chalin 🤩

chalin · 2023-08-16T14:30:47Z

@julianocosta89 - if this works for you, could you approve the PR? Thanks.

julianocosta89 · 2023-08-16T14:53:16Z

I'm on my mobile ATM, but I can take a look tomorrow if that's fine.

chalin · 2023-08-16T15:33:40Z

Thanks, no problem. @cartermp might get to reviewing it before then.

julianocosta89 · 2023-08-17T04:49:22Z

Added the error message here: #3187 (comment)

chalin requested a review from a team August 16, 2023 13:48

chalin mentioned this pull request Aug 16, 2023

CI improvements - 2023/08 #3122

Closed

12 tasks

chalin force-pushed the chalin-im-json-pp-2023-08-16 branch from 9529d28 to ebf5c4a Compare August 16, 2023 14:16

cartermp approved these changes Aug 16, 2023

View reviewed changes

[CI] Use Perl Core module JSON::PP

fd78d6d

chalin force-pushed the chalin-im-json-pp-2023-08-16 branch from ebf5c4a to fd78d6d Compare August 16, 2023 22:30

chalin merged commit 20230e6 into open-telemetry:main Aug 16, 2023

chalin deleted the chalin-im-json-pp-2023-08-16 branch August 16, 2023 22:34

Provide feedback