🐛 Support dashed - generation.

[mjudeikis]

Summary

We don't honor groupGoName and PacakgeName in our generator (different bug somewhere). So when generating clients for things like kube-bind.io this gets crazy. This is hack, and we should fix this properly. But as - values are not supported either way in aliasing and methodNames, I don't feel that bad doing this hack.

If this works (gonna test and update this message), I propose we merge and create a follow-up issue to get to the bottom of where we lose marker value :/ This is ka inda stop-gap solution for now.

Related issue(s)

Fixes #

Release Notes

HACK: Fix `-` dashed package generation by force replace `-` 

[kcp-ci-bot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please ask for approval from mjudeikis. For more information see the Kubernetes Code Review Process.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[kcp-ci-bot]

@mjudeikis: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
pull-code-generator-test	e56c97e	link	true	/test pull-code-generator-test
pull-code-generator-lint	e56c97e	link	true	/test pull-code-generator-lint
pull-code-generator-verify	e56c97e	link	true	/test pull-code-generator-verify

Full PR test history

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[mjudeikis]

replaced by #97