Closed jaredallard closed 3 weeks ago
Latest commit: |
1bd6d0e
|
Status: | ✅ Deploy successful! |
Preview URL: | https://c82d6f80.stencil-1wj.pages.dev |
Branch Preview URL: | https://jaredallard-refactor-modules-eg45.stencil-1wj.pages.dev |
Attention: Patch coverage is 49.05660% with 27 lines in your changes missing coverage. Please review. |
Files | Patch % | Lines |
---|---|---|---|
internal/modules/nativeext/nativeext.go | 37.14% | 19 Missing and 3 partials :warning: | |
internal/modules/module.go | 60.00% | 3 Missing and 1 partial :warning: | |
internal/codegen/tpl.go | 0.00% | 0 Missing and 1 partial :warning: |
:loudspeaker: Thoughts on this report? Let us know!
This needs updated docs for native extensions
The public interface is still the same, I just scanned the docs and didn't see anything :thinking:
Renames
extensions
tonativeext
and starts cleaning up the module resolving logic to be less all over the place and to better integrate the native extensions code with the rest of the module resolving logic.This also switches from
go-git
to runninggit
commands directly to ease the system authentication story.