Skip to content
This repository has been archived by the owner on Apr 23, 2024. It is now read-only.

Migrate to cue.Context() #18

Open
Mitsuwa opened this issue Oct 13, 2023 · 2 comments
Open

Migrate to cue.Context() #18

Mitsuwa opened this issue Oct 13, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@Mitsuwa
Copy link
Collaborator

Mitsuwa commented Oct 13, 2023

What problem are you facing?

this issue is for tracking migrating this function to use cue.Context() instead of the current implementation of cueCompile

https://pkg.go.dev/cuelang.org/go/cue@v0.6.0#Context

How could this Function help solve your problem?

This will allow the base Encode() and the Compile to function off of cue's core apis that are being built currently

@Mitsuwa Mitsuwa added the enhancement New feature or request label Oct 13, 2023
@Mitsuwa
Copy link
Collaborator Author

Mitsuwa commented Oct 13, 2023

the current implementaton uses https://pkg.go.dev/cuelang.org/go/cue/load@v0.6.0#Config which is actually quite extensible for our use case, so perhaps this may not need to be done

@Mitsuwa Mitsuwa closed this as completed Oct 15, 2023
@Mitsuwa
Copy link
Collaborator Author

Mitsuwa commented Dec 18, 2023

Reopening this after speaking with @myitcv

@Mitsuwa Mitsuwa reopened this Dec 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant