Skip to content

chore: remove stale TODOs in rinterface_extra.c#2601

Merged
krlmlr merged 3 commits intomainfrom
copilot/check-todos-in-rinterface-extra-c
Apr 9, 2026
Merged

chore: remove stale TODOs in rinterface_extra.c#2601
krlmlr merged 3 commits intomainfrom
copilot/check-todos-in-rinterface-extra-c

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Mar 27, 2026

Two stale TODOs existed in rinterface_extra.c:

  • /* TODO: serialization */ — placeholder comments in fully-implemented attribute getter functions (string/numeric/bool getters for graph, vertex, and edge attributes). No serialization work is pending; removed.
  • igraph_i_graphlets_project TODO — already resolved in a prior change; rinterface.c now calls the public igraph_graphlets_project API. No action needed.
  • By submitting this pull request, I assign the copyright of my contribution to The igraph development team.

💬 Send tasks to Copilot coding agent from Slack and Teams to turn conversations into code. Copilot posts an update in your thread when it's finished.

Copilot AI linked an issue Mar 27, 2026 that may be closed by this pull request
…face_extra.c`

Agent-Logs-Url: https://github.com/igraph/rigraph/sessions/cd1653f2-83e5-4b8f-8ad0-45db48c56ae9

Co-authored-by: schochastics <17147355+schochastics@users.noreply.github.com>
Copilot AI changed the title [WIP] Check and resolve TODOs in rinterface_extra.c chore: remove stale TODOs in rinterface_extra.c Mar 27, 2026
Copilot AI requested a review from schochastics March 27, 2026 18:59
@krlmlr krlmlr marked this pull request as ready for review April 9, 2026 09:01
@krlmlr krlmlr enabled auto-merge (squash) April 9, 2026 09:01
@krlmlr krlmlr merged commit 6264cdd into main Apr 9, 2026
5 of 6 checks passed
@krlmlr krlmlr deleted the copilot/check-todos-in-rinterface-extra-c branch April 9, 2026 09:19
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.

TODOs in rinterface_extra.c

3 participants