-
Notifications
You must be signed in to change notification settings - Fork 37
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
af_new_drydock_image #18
af_new_drydock_image #18
Conversation
Security InsightsNo security relevant content was detected by automated scans. Action Items
Questions or Comments? Reach out on HipChat: InfoSec Forum. |
Public API ChangesNo changes to the public API found for commit d2c0aca Showing results for d2c0aca
Last edited UTC Jun 25 at 15:50:11 |
Codecov Report
@@ Coverage Diff @@
## master #18 +/- ##
=======================================
Coverage 87.02% 87.02%
=======================================
Files 3 3
Lines 478 478
=======================================
Hits 416 416
Misses 62 62 Continue to review full report at Codecov.
|
This repo builds on Travis, do we still need this @dustinlessard-wf? |
@michaelcarter-wf Looking at the smithy.yaml and Dockerfile ... this repo does not run any tests with either content-shell or dartium. So, we could just close this PR or just merge it in. If someone in the future decides to run tests in Workiva build with the old image, they will get the expired Dartium problem and their tests will hang. |
+1 |
+10 CI passes (moved to Workiva build) |
+10 / +1 |
Purpose
The following pull request has been created by App Frameworks to help move consumers to the latest Smithy image. This was done because of an issue with Dartium and Content-Shell that would fail your CIs. Please seethis Wiki Questionfor additional context.
Changes
The following changes have been made:
drydock-prod.workiva.net/workiva/smithy-runner-generator:355624
.Questions
If you have any questions, please reach out to us in the Support:H5 / Dart Hipchat room