Skip to content

Conversation

@jefferis
Copy link
Collaborator

@jefferis jefferis commented Aug 8, 2025

  • explicit links to all remote packages now required in docs
  • fixes because SWC files are now excluded from tarballs
  • fixes for recent igraph deprecations

jefferis added 7 commits July 28, 2025 14:05
* also remove unnecessary executable permissions
* workaround for new behaviour in R 4.5.1 associated with this change wch/r-source@7bc89c8
* we want to exclude swc.txt files since they are neither swc nor nml
* as.directed, as.undirected, graph
* decided to wrap them so we don't have to fast forward igraph dependency to >=2.1.0 from >=1.0
* had to remove the explicit function imports (because we don't know which to import)
* note that I also had to make an example conditional based on the installed version for the same reasons
* see also igraph/rigraph#1513
@codecov
Copy link

codecov bot commented Aug 8, 2025

Codecov Report

❌ Patch coverage is 60.00000% with 4 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (master@c08b71a). Learn more about missing BASE report.
⚠️ Report is 11 commits behind head on master.

Files with missing lines Patch % Lines
R/ngraph.R 55.55% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##             master     #520   +/-   ##
=========================================
  Coverage          ?   76.94%           
=========================================
  Files             ?       48           
  Lines             ?     6082           
  Branches          ?        0           
=========================================
  Hits              ?     4680           
  Misses            ?     1402           
  Partials          ?        0           

☔ 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.

* there is a problem with Clang 18 on ubuntu and the register keyword.
* this will need to be fixed soon but since it is really beyond the scope of this repo I am going to work around for now
@jefferis jefferis merged commit 2b8342e into master Aug 10, 2025
8 checks passed
@jefferis jefferis deleted the fix/r-4.5.1 branch August 10, 2025 10:44
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