feat(arrow/_examples): enhance library examples#394
Conversation
|
Instead of using the |
|
Some files are missing the Apache license header, adding that should fix the CI |
|
hey i'm super sorry this is taking so long! I've been traveling for a conference. I'll review this and hopefully merge it today |
zeroshade
left a comment
There was a problem hiding this comment.
These are great examples! Thanks so much for this
|
@kris-gaudel looks like you need to update the tinygo test script because you renamed/moved the helloworld/main.go file 😄 that's probably the last thing we need before we can merge this |
|
It looks like there is a memory leak in one of the tests, should I fix it in this PR or make a separate one? |
|
That's a transient leak that I haven't been able to track down myself. If you're able to track it down and fix it, that would be amazing! It can definitely be a separate PR and doesn't need to be dealt with in this one. |
Rationale for this change
Enhancing the library examples
What changes are included in this PR?
3 new examples on how to use the
arrow-golibrary:Are these changes tested?
Not applicable
Are there any user-facing changes?
No, just enhancing examples