Skip to content

Add log message attachment example#1828

Merged
fractalwrench merged 1 commit intofile-attachment-apifrom
file-attachment-example
Jan 29, 2025
Merged

Add log message attachment example#1828
fractalwrench merged 1 commit intofile-attachment-apifrom
file-attachment-example

Conversation

@fractalwrench
Copy link
Member

Goal

Adds a log message attachment sample to the example app. As this relies on a new API it won't compile without a local artefact for now, so I'll avoid merging this until the feature has shipped.

@fractalwrench fractalwrench requested a review from a team as a code owner January 17, 2025 07:14
@fractalwrench fractalwrench force-pushed the file-attachment-example branch from 9f2e056 to e0372a7 Compare January 27, 2025 14:08
@codecov
Copy link

codecov bot commented Jan 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.37%. Comparing base (123496d) to head (27060c1).
Report is 5 commits behind head on file-attachment-api.

Additional details and impacted files

Impacted file tree graph

@@                   Coverage Diff                   @@
##           file-attachment-api    #1828      +/-   ##
=======================================================
- Coverage                85.38%   85.37%   -0.01%     
=======================================================
  Files                      474      474              
  Lines                    11158    11158              
  Branches                  1680     1680              
=======================================================
- Hits                      9527     9526       -1     
+ Misses                     882      881       -1     
- Partials                   749      751       +2     

see 2 files with indirect coverage changes

Copy link
Contributor

@bidetofevil bidetofevil left a comment

Choose a reason for hiding this comment

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

LGTM

@fractalwrench fractalwrench force-pushed the file-attachment-example branch from e0372a7 to 123496d Compare January 29, 2025 11:20
@fractalwrench fractalwrench force-pushed the file-attachment-example branch from 123496d to 27060c1 Compare January 29, 2025 11:45
@fractalwrench fractalwrench merged commit 94eb41d into file-attachment-api Jan 29, 2025
2 of 3 checks passed
@fractalwrench fractalwrench deleted the file-attachment-example branch January 29, 2025 11:45
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.

2 participants