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 3.4.0-rc.1 - use dapr 1.14.0-rc.x #611

Merged
merged 5 commits into from
Aug 1, 2024
Merged

Conversation

DeepanshuA
Copy link
Contributor

@DeepanshuA DeepanshuA commented Jul 26, 2024

Description

Using dapr/dapr 1.14.0-rc.x version.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #[issue number]

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

Signed-off-by: Deepanshu Agarwal <[email protected]>
Signed-off-by: Deepanshu Agarwal <[email protected]>
@DeepanshuA DeepanshuA changed the title [chore] Bump to 3.4.0-rc.1 - use dapr 1.14.0-rc.6 [chore] Bump to 3.4.0-rc.1 - use dapr 1.14.0-rc.5 Jul 26, 2024
@DeepanshuA
Copy link
Contributor Author

Seems same issue that is happening here in e2e test (https://github.com/dapr/js-sdk/actions/runs/10111456256/job/27967538117) has been also reported for java-sdk: dapr/dapr#7938

@mikeee
Copy link
Member

mikeee commented Jul 30, 2024

@DeepanshuA should have been fixed with rc.6 / dapr/dapr#7940
Would it be possible to bump and get this merged?

Co-authored-by: Mike Nguyen <[email protected]>
Signed-off-by: Paul Yuknewicz <[email protected]>
Copy link
Contributor

@paulyuk paulyuk left a comment

Choose a reason for hiding this comment

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

See question on version pls.

@@ -1,6 +1,6 @@
{
"name": "@dapr/dapr",
"version": "3.3.1",
"version": "3.4.0-rc.1",
Copy link
Contributor

Choose a reason for hiding this comment

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

@DeepanshuA are we sticking with 3.3.1 to go with Dapr 1.14.0, or bumping to 3.4? I see that the key bug fix you made for health probe went into 3.3.1.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

js-sdk ver 3.3.1 was released already. We would release 3.4.0 now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Once dapr 1.14 is released finally, will do so. As of now, dapr 1.14 rc versions are out, so creating js-sdk versions in sync with that.

Signed-off-by: Deepanshu Agarwal <[email protected]>
Signed-off-by: Deepanshu Agarwal <[email protected]>
@DeepanshuA DeepanshuA marked this pull request as ready for review August 1, 2024 06:23
@DeepanshuA DeepanshuA requested review from a team as code owners August 1, 2024 06:23
@DeepanshuA
Copy link
Contributor Author

test:e2e is failing intermittenntly. I have tested on local as well, it is passing.
Here, also it passed in this particular run: https://github.com/dapr/js-sdk/actions/runs/10183072399/job/28196930203?pr=611

@DeepanshuA DeepanshuA changed the title [chore] Bump to 3.4.0-rc.1 - use dapr 1.14.0-rc.5 [chore] Bump to 3.4.0-rc.1 - use dapr 1.14.0-rc.x Aug 1, 2024
@yaron2 yaron2 added this pull request to the merge queue Aug 1, 2024
Merged via the queue into main with commit d7547ce Aug 1, 2024
6 checks passed
@marcduiker
Copy link
Contributor

@holopin-bot @DeepanshuA Thanks Deepanshu!

Copy link

holopin-bot bot commented Aug 15, 2024

Congratulations @DeepanshuA, the maintainer of this repository has issued you a badge! Here it is: https://holopin.io/claim/clzvakqzn16210cjybmv9xu12

This badge can only be claimed by you, so make sure that your GitHub account is linked to your Holopin account. You can manage those preferences here: https://holopin.io/account.
Or if you're new to Holopin, you can simply sign up with GitHub, which will do the trick!

paulyuk added a commit that referenced this pull request Aug 17, 2024
* fix: typo (#607)

Update AbstractActor.ts

Signed-off-by: Bond <[email protected]>

* fix: Remove broken image links (#610)

Update README.md

Signed-off-by: Joe Bowbeer <[email protected]>

* Fix README.md for distributedlock example (#613)

Update README.md

Signed-off-by: greenie-msft <[email protected]>

* [chore] Bump to 3.4.0-rc.1 - use dapr 1.14.0-rc.x (#611)

* use dapr 1.14.0-rc.6

Signed-off-by: Deepanshu Agarwal <[email protected]>

* Version correction

Signed-off-by: Deepanshu Agarwal <[email protected]>

* Update .github/workflows/test-e2e.yml

Co-authored-by: Mike Nguyen <[email protected]>
Signed-off-by: Paul Yuknewicz <[email protected]>

* update

Signed-off-by: Deepanshu Agarwal <[email protected]>

* update

Signed-off-by: Deepanshu Agarwal <[email protected]>

---------

Signed-off-by: Deepanshu Agarwal <[email protected]>
Signed-off-by: Paul Yuknewicz <[email protected]>
Co-authored-by: Paul Yuknewicz <[email protected]>
Co-authored-by: Mike Nguyen <[email protected]>

* Updating package json with final release version 3.4.0

Signed-off-by: Paul Yuknewicz <[email protected]>

* Regenerated protos by updating and running ./scripts/fetch-proto.sh

Signed-off-by: Paul Yuknewicz <[email protected]>

* Updating test workflow with correct Dapr Runtime and CLI versions

Signed-off-by: Paul Yuknewicz <[email protected]>

---------

Signed-off-by: Bond <[email protected]>
Signed-off-by: Joe Bowbeer <[email protected]>
Signed-off-by: greenie-msft <[email protected]>
Signed-off-by: Deepanshu Agarwal <[email protected]>
Signed-off-by: Paul Yuknewicz <[email protected]>
Co-authored-by: Bond <[email protected]>
Co-authored-by: Joe Bowbeer <[email protected]>
Co-authored-by: greenie-msft <[email protected]>
Co-authored-by: Deepanshu Agarwal <[email protected]>
Co-authored-by: Mike Nguyen <[email protected]>
@paulyuk paulyuk mentioned this pull request Aug 18, 2024
3 tasks
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.

5 participants