10000 include json packages by robbecker-wf · Pull Request #11 · Workiva/w_dio · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
This repository was archived by the owner on Jan 17, 2025. It is now read-only.

include json packages #11

Merged
merged 4 commits into from
Jul 31, 2023
Merged

include json packages #11

merged 4 commits into from
Jul 31, 2023

Conversation

robbecker-wf
Copy link
Member
@robbecker-wf robbecker-wf commented Jul 30, 2023

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

  • renamed to json_serializable_3_5_1
  • changed the dependency on oss json_annotation to the internal published one

Testing

  • ensure that the 2 new artifacts are included in the build artifacts and that they are the right packages and names.

@aviary2-wf
Copy link

Security Insights

No security relevant content was detected by automated scans.

Action Items

  • Review PR for security impact; comment "security review required" if needed or unsure
  • Verify aviary.yaml coverage of security relevant code

Questions or Comments? Reach out on Slack: #support-infosec.

@@ -0,0 +1,13 @@
name: json_annotation_3_1_1
Copy link
Member Author

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
Copy link
Member Author

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
Copy link
Member Author

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
Copy link
Member Author

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

@robbecker-wf
Copy link
Member Author

QA+1 CI passes
@Workiva/release-management-p let's publish em!

Copy link
@rmconsole-wf rmconsole-wf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 from RM

@rm-astro-wf rm-astro-wf merged commit e51f25e into master Jul 31, 2023
@rm-astro-wf rm-astro-wf deleted the add_json_serializable branch July 31, 2023 18:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0