Skip to content
This repository was archived by the owner on Jul 31, 2023. It is now read-only.

Conversation

@erikpetersen-wf
Copy link
Member

Description of Changes

Proposed Testing Steps (If Applicable)

  • Commented out the line when running make run-docker in the client service and everything worked fine.

Relevant Issues (If Applicable)

  • We shouldn't really maintain multiple config files. We should seek to deprecate/remove the catalog config contained within eva, and opt to use the config contained in the catalog solely.

Maintainer Notifications

…igs (such as the local-catalog-config.end in eva-catalog).
@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.

Copy link

@alexalegre-wf alexalegre-wf left a comment

Choose a reason for hiding this comment

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

LGTM

@erikpetersen-wf
Copy link
Member Author

@Workiva/release-management-pp

@rmconsole2-wf rmconsole2-wf merged commit 1a3df1c into master Jul 29, 2019
@rmconsole2-wf rmconsole2-wf deleted the fix-docker-compose branch July 29, 2019 18:02
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.

6 participants