-
Notifications
You must be signed in to change notification settings - Fork 610
Closed
Labels
Description
Search before asking
- I had searched in the issues and found no similar feature requirement.
Description
PR #3640 merges 5–6 go.mod files into one to make maintenance easier. However, it also hurts the UX for those who import the KubeRay CRD.
@andrewsykim suggests:
However, I also think we should not break package compatibiltiy where possible. I think keeping two Go modules for now is a good middle ground:
- Root go.mod
- ray-operator/go.mod -> preserves minimal Go modules for existing projects that import the KubeRay CRD types
Use case
No response
Related issues
No response
Are you willing to submit a PR?
- Yes I am willing to submit a PR!