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

sqeleton vendoring PoC #480

Closed
wants to merge 1 commit into from
Closed

sqeleton vendoring PoC #480

wants to merge 1 commit into from

Conversation

vvkh
Copy link
Contributor

@vvkh vvkh commented Apr 5, 2023

Basically

mkdir vendor
cd vendor
git clone git@github.com:datafold/sqeleton.git
rm -rf sqeletion/.git
rm -rf sqeleton/.github

+ updates in the pyproject.toml file

@vvkh vvkh force-pushed the sqeletion-vendoring branch from 7d5eb08 to d235360 Compare April 5, 2023 14:58
@erezsh
Copy link
Contributor

erezsh commented Apr 5, 2023

What is the point of this change?

@vvkh vvkh force-pushed the sqeletion-vendoring branch from d235360 to 3c9ac15 Compare April 5, 2023 15:57
@vvkh
Copy link
Contributor Author

vvkh commented Apr 5, 2023

@erezsh we briefly discussed the idea with @wvanbergen with the primary goal of improving time to market and simplifying the release process.

@vvkh vvkh requested a review from nolar April 5, 2023 19:18
@wvanbergen
Copy link

Because we shelved our plans for the other tool that would use the sqeleton library, it doesn't really make much sense to have it in a separate repository anymore. It makes contributing to datadiff harder than it has to be, and makes releases a more complicated process. So we decided to merge these repositories back into a single codebase.

Copy link
Contributor

@nolar nolar left a comment

Choose a reason for hiding this comment

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

Thank you for taking care of it. Luckily, we have no rush on this merging, so I suggest that we take a different route — the one that preserves the git history of changes and embeds sqeleton as one extra top-level package into the library — instead of bringing it as a vendored subdirectory which requires import-path injections.

Essentially, that is git remote add sqeleton … + git merge sqeleton/master --allow-unrelated-histories, and then resolve the conflicts on the same-named files (mostly CI & metadata). I am already working on it.

sqeleton = "0.0.8"
sqeleton = { path = "./vendor/sqeleton", develop = true }
Copy link
Contributor

Choose a reason for hiding this comment

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

On a side note: I wonder, will this approach also package the files properly into tarballs & wheels and install it in a clean environment that is not git-based (i.e. pip install data-diff)?

@nolar
Copy link
Contributor

nolar commented Apr 6, 2023

Here is an alternative proposal with the git history preserved: #485

@williebsweet
Copy link
Contributor

@nolar @wvanbergen Should we close this?

@vvkh vvkh closed this Apr 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants