Skip to content

ScopeConfig for pydevlake#5345

Merged
klesh merged 3 commits intoapache:mainfrom
merico-ai:5159_ScopeConfig_pydevlake
Jun 7, 2023
Merged

ScopeConfig for pydevlake#5345
klesh merged 3 commits intoapache:mainfrom
merico-ai:5159_ScopeConfig_pydevlake

Conversation

@CamilleTeruel
Copy link
Copy Markdown
Contributor

Summary

Rename TransformationRule to ScopeConfig that also includes domain types.

Does this close any open issues?

Part of #5159

@CamilleTeruel CamilleTeruel force-pushed the 5159_ScopeConfig_pydevlake branch from 2a13a15 to 3aaf65f Compare June 1, 2023 15:18
@CamilleTeruel CamilleTeruel requested a review from klesh June 1, 2023 15:19
@klesh
Copy link
Copy Markdown
Contributor

klesh commented Jun 2, 2023

@CamilleTeruel Seems like the test failed, can you fix them first? Thanks.

@CamilleTeruel CamilleTeruel force-pushed the 5159_ScopeConfig_pydevlake branch from 3aaf65f to 2c2a2a7 Compare June 2, 2023 13:32
@CamilleTeruel CamilleTeruel force-pushed the 5159_ScopeConfig_pydevlake branch 2 times, most recently from 2a8dbf1 to cc26115 Compare June 6, 2023 14:41
@CamilleTeruel CamilleTeruel marked this pull request as ready for review June 6, 2023 14:43
@CamilleTeruel CamilleTeruel force-pushed the 5159_ScopeConfig_pydevlake branch from cc26115 to 3941371 Compare June 6, 2023 15:23
Comment thread backend/test/helper/api.go Outdated
Comment thread backend/test/helper/api.go Outdated
@CamilleTeruel CamilleTeruel force-pushed the 5159_ScopeConfig_pydevlake branch 6 times, most recently from 0c30b0b to 549ed88 Compare June 6, 2023 22:13
@keon94
Copy link
Copy Markdown
Contributor

keon94 commented Jun 7, 2023

There are some merge conflicts, otherwise LGTM.

keon94
keon94 previously approved these changes Jun 7, 2023
@CamilleTeruel CamilleTeruel force-pushed the 5159_ScopeConfig_pydevlake branch 6 times, most recently from 929e173 to 926e204 Compare June 7, 2023 08:22
Camille Teruel added 3 commits June 7, 2023 10:44
@CamilleTeruel CamilleTeruel force-pushed the 5159_ScopeConfig_pydevlake branch from 926e204 to 5259b91 Compare June 7, 2023 08:44
Copy link
Copy Markdown
Contributor

@klesh klesh left a comment

Choose a reason for hiding this comment

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

LGTM

@klesh klesh merged commit 0c9311e into apache:main Jun 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants