Skip to content

Conversation

Czaki
Copy link
Collaborator

@Czaki Czaki commented Sep 2, 2024

References and relevant issues

actions/upload-artifact#602
https://napari.zulipchat.com/#narrow/stream/348229-randoms/topic/actions.2Fdownload-artifact/near/467017518

Description

Someone forget what semantic versioning means, so we need to try to deal with it. Hopefully found all places.

@Czaki Czaki added the maintenance PR with maintance changes, label Sep 2, 2024
@Czaki Czaki added this to the 0.5.4 milestone Sep 2, 2024
@Czaki Czaki requested a review from a team as a code owner September 2, 2024 20:26
@jni
Copy link
Member

jni commented Sep 2, 2024

oh noooooo actionlint needs to be updated to include the argument lmfaoooooo 🤣

@jni
Copy link
Member

jni commented Sep 2, 2024

.github/workflows/reusable_pip_test.yml:62:11: input "include-hidden-files" is not defined in action "actions/upload-artifact@v4". available inputs are "compression-level", "if-no-files-found", "name", "overwrite", "path", "retention-days" [action]

@Czaki
Copy link
Collaborator Author

Czaki commented Sep 2, 2024

Let's see if actionlint maintainers response fast.

@Czaki Czaki changed the title Fix coderage and debug upload Fix coverage and debug upload Sep 2, 2024
Copy link

codecov bot commented Sep 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.73%. Comparing base (ff0ec52) to head (82c2641).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #7241      +/-   ##
==========================================
- Coverage   92.82%   92.73%   -0.10%     
==========================================
  Files         623      623              
  Lines       57044    57044              
==========================================
- Hits        52953    52900      -53     
- Misses       4091     4144      +53     

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

@Czaki Czaki added the ready to merge Last chance for comments! Will be merged in ~24h label Sep 3, 2024
@Czaki Czaki changed the title Fix coverage and debug upload Fix coverage and debug artifacts upload Sep 3, 2024
@jni jni merged commit d27de6a into napari:main Sep 3, 2024
42 checks passed
@jni jni deleted the fix_coverageupload branch September 3, 2024 18:12
@github-actions github-actions bot removed the ready to merge Last chance for comments! Will be merged in ~24h label Sep 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance PR with maintance changes,
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants