Skip to content

Conversation

itaigilo
Copy link
Contributor

No description provided.

@itaigilo itaigilo requested review from a team, itaiad200 and nopcoder September 23, 2025 13:58
@itaigilo itaigilo added exclude-changelog PR description should not be included in next release changelog minor-change Used for PRs that don't require issue attached labels Sep 23, 2025
Copy link
Contributor

@nopcoder nopcoder left a comment

Choose a reason for hiding this comment

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

lgtm, I think we should add the fix for s3 update

Copy link
Collaborator

@ozkatz ozkatz left a comment

Choose a reason for hiding this comment

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

I believe this PR accidentally mixes up Enterprise and OSS changes

CHANGELOG.md Outdated

:new: What's new:

- Iceberg REST catalog transactions use hidden branches (#1078)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is a lakeFS Enterprise change, I believe we only advertise OSS changes here.

Copy link
Contributor Author

@itaigilo itaigilo Sep 23, 2025

Choose a reason for hiding this comment

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

Indeed, thanks.
Updated.

@itaigilo itaigilo requested a review from ozkatz September 23, 2025 16:44
Copy link
Collaborator

@ozkatz ozkatz left a comment

Choose a reason for hiding this comment

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

thx!!

@itaigilo itaigilo merged commit 05f0b82 into master Sep 23, 2025
40 checks passed
@itaigilo itaigilo deleted the release/changelog-v1_70_1 branch September 23, 2025 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

exclude-changelog PR description should not be included in next release changelog minor-change Used for PRs that don't require issue attached

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants