Skip to content

[makefile] add controllerGen option allowDangerousTypes#470

Open
notandy wants to merge 1 commit intomainfrom
controllerGen_allowDangerousTypes
Open

[makefile] add controllerGen option allowDangerousTypes#470
notandy wants to merge 1 commit intomainfrom
controllerGen_allowDangerousTypes

Conversation

@notandy
Copy link
Contributor

@notandy notandy commented Mar 16, 2026

This PR adds allowDangerousTypes to the controllerGen option, which is
defined as 1:

`allowDangerousTypes bool`
allows types which are usually omitted from CRD generation
because they are not recommended.

Currently the following additional types are allowed when this is true:
float32 float64

Left unspecified, the default is false

Required by a developer who want's to use floating points in the Spec.

This PR adds allowDangerousTypes to the controllerGen option, which is
defined as [1]:

    `allowDangerousTypes bool`
    allows types which are usually omitted from CRD generation
    because they are not recommended.

    Currently the following additional types are allowed when this is true:
    float32 float64

    Left unspecified, the default is false

[1]: https://book.kubebuilder.io/reference/controller-gen.html

Required by a develop who want's to use floating points in the Spec.
@notandy notandy requested a review from a team as a code owner March 16, 2026 16:07
@github-actions
Copy link
Contributor

Merging this branch will decrease overall coverage

Impacted Packages Coverage Δ 🤖
github.com/sapcc/go-makefile-maker/internal/core 0.00% (ø)
github.com/sapcc/go-makefile-maker/internal/makefile 7.07% (-0.06%) 👎

Coverage by file

Changed files (no unit tests)

Changed File Coverage Δ Total Covered Missed 🤖
github.com/sapcc/go-makefile-maker/internal/core/config.go 0.00% (ø) 54 0 54
github.com/sapcc/go-makefile-maker/internal/makefile/makefile.go 0.00% (ø) 240 (+3) 0 240 (+3)

Please note that the "Total", "Covered", and "Missed" counts above refer to code statements instead of lines of code. The value in brackets refers to the test coverage of that file in the old version of the code.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant