Skip to content

Adding logos#368

Merged
jesper-friis merged 15 commits intomasterfrom
tripper_logo
Mar 28, 2025
Merged

Adding logos#368
jesper-friis merged 15 commits intomasterfrom
tripper_logo

Conversation

@jesper-friis
Copy link
Contributor

Description

Adding the logos created by @francescalb

Type of change

  • Bug fix and code cleanup
  • New feature
  • Documentation update
  • Testing

Checklist for the reviewer

This checklist should be used as a help for the reviewer.

  • Is the change limited to one issue?
  • Does this PR close the issue?
  • Is the code easy to read and understand?
  • Do all new feature have an accompanying new test?
  • Has the documentation been updated as necessary?
  • Is the code properly tested?

@codecov
Copy link

codecov bot commented Mar 27, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 80.43%. Comparing base (61e226b) to head (36fa4b9).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #368      +/-   ##
==========================================
- Coverage   80.44%   80.43%   -0.01%     
==========================================
  Files          25       25              
  Lines        2971     2970       -1     
==========================================
- Hits         2390     2389       -1     
  Misses        581      581              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

README.md Outdated
@@ -1,3 +1,5 @@
<img src="https://raw.githubusercontent.com/EMMC-ASBL/tripper/refs/heads/tripper_logo/docs/figs/tripper_logo.svg" alt="Tripper logo" align="right" width="72" height="72">
Copy link
Contributor

@francescalb francescalb Mar 28, 2025

Choose a reason for hiding this comment

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

This will not resolve correctly once the PR is merged

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Very good point. The links are now updated to master

docs/index.md Outdated
@@ -1,3 +1,5 @@
<img src="https://raw.githubusercontent.com/EMMC-ASBL/tripper/refs/heads/tripper_logo/figs/tripper_logo.svg" alt="Tripper logo" align="right" width="72" height="72">
Copy link
Contributor

Choose a reason for hiding this comment

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

Wrong path, also within this PR.

Copy link
Contributor

@francescalb francescalb left a comment

Choose a reason for hiding this comment

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

See my comments

jesper-friis and others added 2 commits March 28, 2025 11:41
Co-authored-by: Francesca L. Bleken <48128015+francescalb@users.noreply.github.com>
@jesper-friis jesper-friis merged commit aabd5ec into master Mar 28, 2025
19 checks passed
@jesper-friis jesper-friis deleted the tripper_logo branch March 28, 2025 14:24
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.

2 participants