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

v0.110.0 change log #6605

Merged
merged 7 commits into from
Sep 14, 2023
Merged

v0.110.0 change log #6605

merged 7 commits into from
Sep 14, 2023

Conversation

idanovo
Copy link
Contributor

@idanovo idanovo commented Sep 14, 2023

No description provided.

@idanovo idanovo added the exclude-changelog PR description should not be included in next release changelog label Sep 14, 2023
@github-actions
Copy link

No linked issues found. Please add the corresponding issues in the pull request description.
Use GitHub automation to close the issue when a PR is merged

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.

The bug fixes section is missing.
Need to list the deprecated API in this release in a clear way.

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.

last comment should set a request for change

@idanovo idanovo requested a review from nopcoder September 14, 2023 11:54
@idanovo idanovo added the minor-change Used for PRs that don't require issue attached label Sep 14, 2023
CHANGELOG.md Outdated
:new: What's new:
- Restrict lakectl local to common prefixes only (#6510)
- Added URL parser to lua runtime (#6597)
- lakectl: deprecate dbt and metastore commands (#6565)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- lakectl: deprecate dbt and metastore commands (#6565)
- lakectl: dbt and metastore commands marked as deprecated (#6565)

CHANGELOG.md Outdated
- Restrict lakectl local to common prefixes only (#6510)
- Added URL parser to lua runtime (#6597)
- lakectl: deprecate dbt and metastore commands (#6565)
- API: move specific API to internal tag (#6582)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- API: move specific API to internal tag (#6582)
- API: Update APIs tag for internal use (#6582)

@@ -2,9 +2,19 @@

## UNRELEASED

# v0.110.0

:new: What's new:
Copy link
Contributor

Choose a reason for hiding this comment

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

would group all the deprecated api to a single group

CHANGELOG.md Outdated
- lakectl: deprecate dbt and metastore commands (#6565)
- API: move specific API to internal tag (#6582)
- Upgrade code to use AWS SDK go v2 (#6486)
- Deprecate: remove expand template API (#6587)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Deprecate: remove expand template API (#6587)
- Remove deprecated APIs: expand template (#6587)

CHANGELOG.md Outdated
- API: move specific API to internal tag (#6582)
- Upgrade code to use AWS SDK go v2 (#6486)
- Deprecate: remove expand template API (#6587)
- Add URL parser to lua runtime (#6597)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Add URL parser to lua runtime (#6597)
- Actions: Lua package to parse URL (#6597)

CHANGELOG.md Outdated
- Upgrade code to use AWS SDK go v2 (#6486)
- Deprecate: remove expand template API (#6587)
- Add URL parser to lua runtime (#6597)
- Deprecate: remove the update and forgot password APIs (#6591)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Deprecate: remove the update and forgot password APIs (#6591)
- Remove deprecated APIs: update and forgot password (#6591)

CHANGELOG.md Outdated

:bug: Bug fixed:
- Fix lakectl local verify bad path error on Windows (#6602)
- Fix Azure pre-signed URL for blob from different account (#6594)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Fix Azure pre-signed URL for blob from different account (#6594)
- Fix Azure pre-signed URL for blob imported from different account (#6594)

@idanovo idanovo requested a review from nopcoder September 14, 2023 12:24
CHANGELOG.md Outdated
Comment on lines 8 to 9
- Removed and detracted: expand template API- (#6587)
- Removed and detracted: remove the update and forgot password APIs (#6591)
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- Removed and detracted: expand template API- (#6587)
- Removed and detracted: remove the update and forgot password APIs (#6591)
- Removed deprecated: expand template API (#6587)
- Removed deprecated: remove the update and forgot password APIs (#6591)

Copy link
Contributor

Choose a reason for hiding this comment

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

Group the API related and changes and move them under the new stuff

@idanovo idanovo merged commit c85a71c into master Sep 14, 2023
@idanovo idanovo deleted the v0.110.0-release-notes branch September 14, 2023 12:53
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.

2 participants