Skip to content
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

Merged

Conversation

dustinlessard-wf
Copy link

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:

  • All instances of the production Smithy image have been updated to 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

@aviary-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 HipChat: InfoSec Forum.

@semveraudit-wf
Copy link

semveraudit-wf commented Jun 22, 2018

Public API Changes

No changes to the public API found for commit d2c0aca

Showing results for d2c0aca

Powered by semver-audit-service. Please report any problems by filing an issue.
Reported by semver-audit-dart 1.4.0
Browse public API.

Last edited UTC Jun 25 at 15:50:11

@codecov-io
Copy link

Codecov Report

Merging #18 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           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.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f4ac8f2...d2c0aca. Read the comment docs.

@michaelcarter-wf
Copy link

This repo builds on Travis, do we still need this @dustinlessard-wf?

@robbecker-wf
Copy link
Member

@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.

@robbecker-wf
Copy link
Member

+1

@robbecker-wf
Copy link
Member

+10 CI passes (moved to Workiva build)

@michaelcarter-wf
Copy link

+10 / +1

@michaelcarter-wf michaelcarter-wf merged commit f3d8875 into Workiva:master Jun 25, 2018
rm-astro-wf pushed a commit that referenced this pull request Jan 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants