Skip to content

test: merged and refactored indexing.R tests#1687

Merged
schochastics merged 4 commits intoigraph:mainfrom
schochastics:test-indexing
Feb 19, 2025
Merged

test: merged and refactored indexing.R tests#1687
schochastics merged 4 commits intoigraph:mainfrom
schochastics:test-indexing

Conversation

@schochastics
Copy link
Copy Markdown
Contributor

@schochastics schochastics commented Feb 18, 2025

also replaced stop with cli::cli_abort anbd renamed canonicalize_matrix to as_unnamed_dense_matrix

ref #1688

@aviator-app
Copy link
Copy Markdown
Contributor

aviator-app bot commented Feb 18, 2025

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged manually (without Aviator). Merging manually can negatively impact the performance of the queue. Consider using Aviator next time.


See the real-time status of this PR on the Aviator webapp.
Use the Aviator Chrome Extension to see the status of your PR within GitHub.

maelle
maelle previously requested changes Feb 19, 2025
Copy link
Copy Markdown
Contributor

@maelle maelle left a comment

Choose a reason for hiding this comment

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

Thank you!

schochastics and others added 2 commits February 19, 2025 12:10
Co-authored-by: Maëlle Salmon <maelle.salmon@yahoo.se>
@schochastics schochastics dismissed maelle’s stale review February 19, 2025 11:50

requested change was made

@schochastics schochastics merged commit 9cc5103 into igraph:main Feb 19, 2025
22 checks passed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 20, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants