🚇🧹 Switch back to GeekyEggo/delete-artifact

[s-weigand]

Switch back to GeekyEggo/delete-artifact after the permission issue fix was released.

Change summary

• 🚇🧹 Switch back to GeekyEggo/delete-artifact

Checklist

• ✔️ Passing the tests (mandatory for all PR's)

[github-actions]

👈 Launch a binder notebook on branch s-weigand/pyglotaran/switch-back-delete-artifact

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

[sourcery-ai]

Hey @s-weigand - I've reviewed your changes and they look great!

General suggestions:

• Confirm that GeekyEggo/delete-artifact@v5 includes all necessary fixes and enhancements for your workflow requirements.
• Consider adding a brief comment next to the action usage to document the reason for the switch back for future reference.

Here's what I looked at during the review

• 🟡 General issues: 1 issue found
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Docstrings: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.}

[github-actions]

Benchmark is done. Checkout the benchmark result page.
Benchmark differences below 5% might be due to CI noise.

Benchmark diff v0.7.2 vs. main

All benchmarks:

| Change   | Before [1fb14bbc] <v0.7.2>   | After [f95130fd]    |   Ratio | Benchmark (Parameter)                   |
|----------|------------------------------|---------------------|---------|-----------------------------------------|
|          | 241M                         | 241M                |    1    | IntegrationTwoDatasets.peakmem_optimize |
|          | 1.38±0.03s                   | 1.49±0.1s           |    1.08 | IntegrationTwoDatasets.time_optimize    |

Benchmark diff main vs. PR

All benchmarks:

| Change   | Before [f95130fd]    | After [f7e6a218]    |   Ratio | Benchmark (Parameter)                   |
|----------|----------------------|---------------------|---------|-----------------------------------------|
|          | 241M                 | 241M                |    1    | IntegrationTwoDatasets.peakmem_optimize |
|          | 1.49±0.1s            | 1.40±0.08s          |    0.94 | IntegrationTwoDatasets.time_optimize    |

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.6%. Comparing base (2c88dce) to head (f7e6a21).
Report is 2 commits behind head on main.

Additional details and impacted files

@@          Coverage Diff          @@
##            main   #1441   +/-   ##
=====================================
  Coverage   88.6%   88.6%           
=====================================
  Files        107     107           
  Lines       5128    5133    +5     
  Branches     962     965    +3     
=====================================
+ Hits        4544    4549    +5     
  Misses       468     468           
  Partials     116     116           

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

[jsnel]

V5 of action as expected