Skip to content

refactor: Replace loop by lapply() when returning vertex and edge sets #1033

Merged
aviator-app[bot] merged 1 commit intomainfrom
f-lapply-vs-es
Dec 29, 2023
Merged

refactor: Replace loop by lapply() when returning vertex and edge sets #1033
aviator-app[bot] merged 1 commit intomainfrom
f-lapply-vs-es

Conversation

@krlmlr
Copy link
Copy Markdown
Contributor

@krlmlr krlmlr commented Dec 29, 2023

Prompted by #1028.

@aviator-app
Copy link
Copy Markdown
Contributor

aviator-app bot commented Dec 29, 2023

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 using Aviator.


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.

@krlmlr krlmlr changed the title chore: Regenerate parser refactor: Replace loop by lapply() when returning vertex and edge sets Dec 29, 2023
@krlmlr krlmlr changed the title refactor: Replace loop by lapply() when returning vertex and edge sets refactor: Replace loop by lapply() when returning vertex and edge sets Dec 29, 2023
@krlmlr krlmlr requested a review from szhorvat December 29, 2023 04:44
@krlmlr
Copy link
Copy Markdown
Contributor Author

krlmlr commented Dec 29, 2023

Do the names of the return values in the affected functions make sense to you?

@aviator-app aviator-app bot merged commit c2fae10 into main Dec 29, 2023
@aviator-app aviator-app bot deleted the f-lapply-vs-es branch December 29, 2023 05:29
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 29, 2024
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.

1 participant