Skip to content

Conversation

@indietyp
Copy link
Member

@indietyp indietyp commented Feb 8, 2023

🌟 What is the purpose of this PR?

This was an oversight in #1980, where we removed the leading space from attachments but forgot multiline attachments.

🔗 Related links

@indietyp indietyp requested a review from a team as a code owner February 8, 2023 10:10
@github-actions github-actions bot added the area/libs > error-stack Affects the `error-stack` crate (library) label Feb 8, 2023
@codecov
Copy link

codecov bot commented Feb 8, 2023

Codecov Report

Merging #2022 (729f972) into main (89d4f9c) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #2022   +/-   ##
=======================================
  Coverage   56.14%   56.14%           
=======================================
  Files         270      270           
  Lines       19574    19574           
  Branches      421      421           
=======================================
  Hits        10990    10990           
  Misses       8579     8579           
  Partials        5        5           
Flag Coverage Δ
backend-integration-tests 3.66% <ø> (ø)
error-stack 82.54% <100.00%> (ø)
unit-tests 1.49% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
libs/error-stack/src/fmt.rs 96.36% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@indietyp
Copy link
Member Author

indietyp commented Feb 8, 2023

/trunk merge

@trunk-io trunk-io bot merged commit c890b0f into main Feb 8, 2023
@trunk-io trunk-io bot deleted the bm/es/space branch February 8, 2023 15:48
@trunk-io
Copy link

trunk-io bot commented Feb 8, 2023

😎 Merged successfully (details).

trunk-io bot pushed a commit that referenced this pull request Feb 8, 2023
## 🌟 What is the purpose of this PR?

A small bug was highlighted in #2019 and fixed in #2022, so we want to publish the fix.

## 🔗 Related links

- Closes #2019
- #2022

## ⚠️ Known issues

Currently, the `SemVer Check` job is disabled, so we have to run it manually. Output:
```
❯ cargo semver-checks check-release       
    Updating index
     Parsing error-stack v0.3.1 (current)
     Parsing error-stack v0.3.0 (baseline)
    Checking error-stack v0.3.0 -> v0.3.1 (minor change)
   Completed [   0.127s] 35 checks; 35 passed, 5 unnecessary
cargo semver-checks check-release  11.95s user 2.40s system 87% cpu 16.334 total
❯ echo $?
0
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/libs > error-stack Affects the `error-stack` crate (library)

Development

Successfully merging this pull request may close these issues.

3 participants