Roy J. Wignarajah comments

Results 41 comments of


                                            Roy J. Wignarajah

Extract ListImpl class from List class

I might need a couple days to study the changes in the linked issue and PR but I think I can do this. I would like to try it out.

Extract ListImpl class from List class

Apologies for the delay. I've been preoccupied with school and other commitments. Wanted to give an update on my progress. So far I've only been able to extract the `head_`...

Extract ListImpl class from List class

@gavv Apologies for the radio silence. I couldn't figure out how to extract the remaining methods, even after referring to #593 . Please unassign me. I am however, interested in...

Extract ListImpl class from List class

@veronikaro Thank you for taking this on! I'm excited to learn how you refactor this.

Switch sharing to use remix

@rjwignar looping myself in for awareness. I'd like to work on related issues once this is setup

ci: clang_format.yml lints only modified source files

Leaving draft PR up for now, will finalize the PR tomorrow.

ci: clang_format.yml lints only modified source files

Hi @alandtse thanks for reaching out. I've been meaning to update the glob but I've been bogged down in class commitments. I likely won't have time to work on this...

Clang format action should probably only touch changed files

I'd like to look into this issue.

Clang format action should probably only touch changed files

### The Plan My plan for this was to use another Marketplace job that could retrieve a list of changed files and then pass that list to the clang-format action....

Clang format action should probably only touch changed files

I agree, I'd rather modify just the workflow file. Clang-format supports passing a file of pathnames using the --files argument but the linting action does that in the background (the...