-
Notifications
You must be signed in to change notification settings - Fork 1
Conversation
Security InsightsNo security relevant content was detected by automated scans. Action Items
Questions or Comments? Reach out on Slack: #support-infosec. |
@@ -0,0 +1,13 @@ | |||
name: json_annotation_3_1_1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
renamed the package. This is the only change to the json_annotation package.
@@ -0,0 +1,37 @@ | |||
name: json_serializable_3_5_2 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
renamed the package
hosted: | ||
name: json_annotation_3_1_1 | ||
url: https://pub.workiva.org | ||
version: ^5.3.0 # This version doesn't matter, there's only the 3_1_1 forked version |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
changed the json_annotation dependency to the temporary copied name
WORKDIR /build/json_serializable-3.5.2 | ||
RUN create_publishable_artifact.sh | ||
|
||
ARG BUILD_ARTIFACTS_PUB=/build/dio/pub_package.pub.tgz:/build/json_annotation-3.1.1/pub_package.pub.tgz:/build/json_serializable-3.5.2/pub_package.pub.tgz |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add the new packages as build artifacts
QA+1 CI passes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 from RM
Summary
Include a straight copy of json_annotation at version 3.1.1, but renamed to json_annotation_3_1_1.
Include a straight copy of json_serializable at version 3.5.2, but with minor needed changes for Workiva
Testing