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

Conversation

@grovesy
Copy link
Member

@grovesy grovesy commented Oct 13, 2020

Initial 'DAGs' for DataHelix/DataHub and Synthetic Architecture

focus of the next meeting

Copy link
Member

@mcleo-d mcleo-d left a comment

Choose a reason for hiding this comment

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

There are a few minor grammatical changes that would be nice to fix. However, these won't stop the team from moving forward, or break the build, if merged.

Copy link
Member

@mcleo-d mcleo-d left a comment

Choose a reason for hiding this comment

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

@grovesy - Thanks for making the changes 👍

@mcleo-d
Copy link
Member

mcleo-d commented Oct 13, 2020

Hi @grovesy

I have merged this PR knowing you are the DataHub Lead Maintainer who raised the PR and asked me to formally review.

Many thanks,

James.

@mcleo-d mcleo-d merged commit 63f6408 into master Oct 13, 2020
@mcleo-d mcleo-d deleted the creating-initial-dags branch October 13, 2020 17:06
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.

3 participants