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

chore: Bump to version 0.45.0 to start release process #4088

Merged
merged 6 commits into from
Jan 31, 2024

Conversation

morristai
Copy link
Member

@morristai morristai commented Jan 29, 2024

What's Changed

Added

Changed

Fixed

Docs

CI

Chore

New Contributors

@github-actions github-actions bot requested review from oowl and silver-ymz January 29, 2024 17:29
@suyanhanx suyanhanx marked this pull request as ready for review January 30, 2024 02:30
@suyanhanx suyanhanx marked this pull request as draft January 30, 2024 02:31
@suyanhanx
Copy link
Member

Sorry for my misoperation😥

@@ -1,289 +0,0 @@
crate 0BSD Apache-2.0 Apache-2.0 WITH LLVM-exception BSD-2-Clause BSD-3-Clause BSL-1.0 CC0-1.0 ISC MIT MPL-2.0 OpenSSL Unicode-DFS-2016 Unlicense Zlib
Copy link
Member

Choose a reason for hiding this comment

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

Looks strange. Why this file is cleared?

Copy link
Member Author

Choose a reason for hiding this comment

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

Fixed. (My apologies, I overlooked the [workspace.dependencies] version in Cargo.toml. Since this version doesn't include the patch version, when I searched for the last version, which is 0.44.2, it didn't show up.)

opendal = { version = "0.45", path = "core" }

@Xuanwo
Copy link
Member

Xuanwo commented Jan 30, 2024

Hi, @morristai, please only push tags from the main branch after the PR has been merged.

@morristai
Copy link
Member Author

Hi, @morristai, please only push tags from the main branch after the PR has been merged.

Sorry, my mistake. Can it be reverted?

@Xuanwo
Copy link
Member

Xuanwo commented Jan 30, 2024

Sorry, my mistake. Can it be reverted?

No, we cannot remove a tag that has already been pushed. Fortunately, our build for rc1 failed, so we now need to push an rc.2 tag which will not impact our subsequent steps.

CHANGELOG.md Show resolved Hide resolved
@morristai
Copy link
Member Author

Just FYI: For the unfinished task - Update core/src/docs/upgrade.md if there are breaking changes in the core, I haven't looked into the breaking changes. Please give me some time.

@Xuanwo
Copy link
Member

Xuanwo commented Jan 30, 2024

Just FYI: For the unfinished task - Update core/src/docs/upgrade.md if there are breaking changes in the core, I haven't looked into the breaking changes. Please give me some time.

Let me update it.

@Xuanwo
Copy link
Member

Xuanwo commented Jan 30, 2024

Hi, there are many changes after this PR created. Would you like to re-generate the changelog? Thanks!

@Zheaoli
Copy link
Member

Zheaoli commented Jan 30, 2024

Hi, there are many changes after this PR created. Would you like to re-generate the changelog? Thanks!

Should we create a version branch to lock on the feature and bugfix we need

@Xuanwo
Copy link
Member

Xuanwo commented Jan 30, 2024

Should we create a version branch to lock on the feature and bugfix we need

Too complex for our project, not interested.

@morristai
Copy link
Member Author

Should we also update the Haskell binding version in Cargo.toml? (It seems so, but the guide only mentions updating the version and the tag field of the source-repository in bindings/haskell/opendal.cabal.)

@Xuanwo
Copy link
Member

Xuanwo commented Jan 30, 2024

Should we also update the Haskell binding version in Cargo.toml?

The haskell binding will have separate release processes. Let's pin 0.44.2 for haskell and remove the path in it.

@morristai
Copy link
Member Author

morristai commented Jan 31, 2024

Hi, there are many changes after this PR created. Would you like to re-generate the changelog? Thanks!

Done!

The haskell binding will have separate release processes. Let's pin 0.44.2 for haskell and remove the path in it.

Addressed

@morristai morristai marked this pull request as ready for review January 31, 2024 04:01
Copy link
Member

@Xuanwo Xuanwo left a comment

Choose a reason for hiding this comment

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

Thanks!

@Xuanwo Xuanwo merged commit 7190c07 into apache:main Jan 31, 2024
189 checks passed
@morristai
Copy link
Member Author

@Xuanwo Now I can push v0.45.0-rc.2 tag, right?

@Xuanwo
Copy link
Member

Xuanwo commented Jan 31, 2024

@Xuanwo Now I can push v0.45.0-rc.2 tag, right?

Yes! Please push a tag on main branch 🥰

@morristai morristai deleted the bump-0.45.0 branch January 31, 2024 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants