Skip to content

Rebase #9: table tennis medallion architecture sample onto current master - #18

Closed
TemRevil wants to merge 1 commit into
adidas:masterfrom
TemRevil:feature/rebase-table-tennis-sample
Closed

TemRevil wants to merge 1 commit into
adidas:masterfrom
TemRevil:feature/rebase-table-tennis-sample

Conversation

@TemRevil

Copy link
Copy Markdown

This is a rebase of #9 onto current master, nothing else. That PR was approved back in January 2024 but never merged, and it's now 24 commits behind, which is probably why it's stuck.

I cherry-picked Gabriel's original commit as-is (same author, same files), and checked that add_current_date, rename, with_expressions, custom_transformation, and optimize_dataset still work the same way on current master. No conflicts, no changes to the actual sample.

Feel free to close this in favor of Gabriel just updating his own branch if that's simpler on your end. Either way the credit should go to him, this is his work, I just moved it forward a couple years.

@jmcorreia

Copy link
Copy Markdown
Contributor

Hi @TemRevil , thanks for your contribution! We will try to give it some priority during the following weeks.

@TemRevil

Copy link
Copy Markdown
Author

awe, you're alive?

@jmcorreia

Copy link
Copy Markdown
Contributor

Included as part of release 2.2.1.
Thanks a lot for your contribution and sorry for the time it took us to consider it!

@jmcorreia jmcorreia closed this Oct 29, 2026
@TemRevil

Copy link
Copy Markdown
Author

Thanks for getting it into 2.2.1, and no worries about the wait. If there is anything else in the backlog you would like a hand with, I am glad to pick it up.

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.

2 participants