Skip to content

Conversation

@halspang
Copy link
Contributor

Description

Update refs to release-1.2-rc1

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: endgame

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

@halspang halspang force-pushed the release-1.2 branch 2 times, most recently from fd47f8f to 61fc7fe Compare May 18, 2021 03:45
@codecov
Copy link

codecov bot commented May 18, 2021

Codecov Report

Merging #547 (28fa823) into master (33f793c) will decrease coverage by 0.13%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #547      +/-   ##
============================================
- Coverage     80.65%   80.52%   -0.14%     
+ Complexity      982      981       -1     
============================================
  Files            88       88              
  Lines          3024     3024              
  Branches        331      331              
============================================
- Hits           2439     2435       -4     
- Misses          420      423       +3     
- Partials        165      166       +1     
Impacted Files Coverage Δ Complexity Δ
sdk/src/main/java/io/dapr/utils/Retry.java 50.00% <0.00%> (-25.00%) 1.00% <0.00%> (-1.00%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 33f793c...28fa823. Read the comment docs.

-->

```bash
dapr stop --app-id exception_example
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this removed due to the related change in CLI to exit with error when app id is not found?
dapr/cli@7b69e1a
@halspang
cc @artursouza

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes. With that change, a lot of the examples will finish and close everything successfully. When we attempted to close it we'd get error messages, hence the removal.

@artursouza artursouza merged commit 30c0089 into dapr:master May 19, 2021
artursouza added a commit to artursouza/java-sdk that referenced this pull request May 19, 2021
artursouza added a commit that referenced this pull request May 19, 2021
* Revert "Update refs to release-1.2-rc1 (#547)"

This reverts commit 30c0089.

* Update CLI and daprd to 1.2.0 RCs.
fackee pushed a commit to fackee/java-sdk that referenced this pull request Mar 28, 2022
Signed-off-by: zjx244729 <zjx244729@alibaba-inc.com>
fackee pushed a commit to fackee/java-sdk that referenced this pull request Mar 28, 2022
* Revert "Update refs to release-1.2-rc1 (dapr#547)"

This reverts commit 30c0089.

* Update CLI and daprd to 1.2.0 RCs.

Signed-off-by: zjx244729 <zjx244729@alibaba-inc.com>
fackee pushed a commit to fackee/java-sdk that referenced this pull request Mar 28, 2022
Signed-off-by: zjx244729 <zjx244729@alibaba-inc.com>
fackee pushed a commit to fackee/java-sdk that referenced this pull request Mar 28, 2022
* Revert "Update refs to release-1.2-rc1 (dapr#547)"

This reverts commit 30c0089.

* Update CLI and daprd to 1.2.0 RCs.

Signed-off-by: zjx244729 <zjx244729@alibaba-inc.com>
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