Skip to content

Add builder methods and move NvPublic #316

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 1 commit into from
Jan 25, 2022

Conversation

ionut-arm
Copy link
Member

This commit brings two changes:

  • implementing builder methods on all types that have their own
    ...Builder type, so that users are more easily directed to them in the
    documentation.
  • moving the nv top-level module within the structures module to
    make things a bit more consistent.

Signed-off-by: Ionut Mihalcea ionut.mihalcea@arm.com

@ionut-arm ionut-arm added the enhancement New feature or request label Jan 21, 2022
@ionut-arm ionut-arm self-assigned this Jan 21, 2022
@ionut-arm ionut-arm mentioned this pull request Jan 21, 2022
16 tasks
Copy link
Collaborator

@Superhepper Superhepper left a comment

Choose a reason for hiding this comment

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

You should make the builder functions const were it is possible.

This commit brings two changes:
* implementing `builder` methods on all types that have their own
`...Builder` type, so that users are more easily directed to them in the
documentation.
* moving the `nv` top-level module within the `structures` module to
make things a bit more consistent.

Signed-off-by: Ionut Mihalcea <ionut.mihalcea@arm.com>
@ionut-arm
Copy link
Member Author

You should make the builder functions const were it is possible.

I've now made all the builder methods that I could constant.

Copy link
Collaborator

@Superhepper Superhepper left a comment

Choose a reason for hiding this comment

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

Nice!

Copy link
Collaborator

@wiktor-k wiktor-k left a comment

Choose a reason for hiding this comment

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

Looks straightforward 👍

@ionut-arm ionut-arm merged commit a32bfb1 into parallaxsecond:main Jan 25, 2022
@ionut-arm ionut-arm deleted the builder-method branch January 25, 2022 16:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants