Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

257 whendodepreciation #340

Merged
merged 3 commits into from
Dec 13, 2023
Merged

257 whendodepreciation #340

merged 3 commits into from
Dec 13, 2023

Conversation

labordep
Copy link
Contributor

@labordep labordep commented Nov 1, 2023

No description provided.

@Ducasse
Copy link
Contributor

Ducasse commented Dec 3, 2023

Hi @labordep Pierre can you have a look at the conflicts. I have no idea how to fix them but we should not keep PR with conflicts open too long.

@labordep
Copy link
Contributor Author

labordep commented Dec 3, 2023

There are 702 file changed, this is not normal. It seems like all file are re-encoded, @tinchodias what can I do ?

@labordep
Copy link
Contributor Author

labordep commented Dec 7, 2023

Hi @tinchodias, this problem are due to not using tonel v1 in my image ?
I can reload this branch with the new format.

@tinchodias
Copy link
Collaborator

You had v1, it's okay, but the files you changed had v2 or v3 format, that's a root of the problem. After there are other changes in the target branch and produces conflicts. This is what I understand. There was similar issue with an Enzo PR and I had troubles to merge but Pablo did it, I will try with Iceberg.

Copy link
Collaborator

@tinchodias tinchodias left a comment

Choose a reason for hiding this comment

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

The real change is hidden by the Tonel version noise, but I believe we have to merge it.

@tinchodias tinchodias merged commit 85ee573 into dev-1.0 Dec 13, 2023
6 checks passed
@tinchodias tinchodias deleted the 257-whendodepreciation branch January 29, 2024 13:36
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