Skip to content

[Fix] Add plural naming formats in scaffolding #21

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Jun 8, 2022

Conversation

varshaprasad96
Copy link
Contributor

@varshaprasad96 varshaprasad96 commented Jun 2, 2022

This PR adds plural formats while defining interfaces which is similar to what code-gen does.

This is blocked on #17 and needs to be rebased after it is merged. Please review only the last 2 commits.
It is ready for review, but blocked for merging hence converting it to draft.

Fixes: #19

@varshaprasad96
Copy link
Contributor Author

Blocked by #17

@varshaprasad96 varshaprasad96 marked this pull request as draft June 2, 2022 22:35
@varshaprasad96 varshaprasad96 changed the title Add naming formats [Fix] Add plural naming formats in scaffolding Jun 2, 2022
@varshaprasad96 varshaprasad96 marked this pull request as ready for review June 7, 2022 20:49
@varshaprasad96 varshaprasad96 requested a review from ncdc June 7, 2022 20:53
@varshaprasad96
Copy link
Contributor Author

Rebased this with master. Not sure why CI isn't getting triggered. Closing and reopening the PR.

@stevekuznetsov
Copy link
Contributor

@varshaprasad96 looks like you still need to rebase - perhaps your local copy of master is out of date?

Copy link
Contributor

@stevekuznetsov stevekuznetsov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, let's just get the whitespace for the copyright stanza correct

@ncdc ncdc merged commit 905ec05 into kcp-dev:main Jun 8, 2022
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.

Add support for plural names for types
3 participants