Skip to content
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

Add getters to internal generated types #3650

Merged
merged 1 commit into from
Oct 14, 2020

Conversation

andrewjdawson2016
Copy link
Contributor

This adds getters to internal types. This is needed because we are attempting to move from generated thrift types to internal types and in some places in the code we use thrift accessors. The purpose of these generated getters is to do the same thing that the thrift access methods

@github-actions github-actions bot force-pushed the internal_types_getters branch from 4de56fa to e506159 Compare October 14, 2020 18:04
@coveralls
Copy link

coveralls commented Oct 14, 2020

Coverage Status

Coverage decreased (-2.6%) to 62.528% when pulling 61203b4 on internal_types_getters into ba0eacb on master.

@github-actions github-actions bot force-pushed the internal_types_getters branch 2 times, most recently from 48dc0dc to 61203b4 Compare October 14, 2020 18:56
@github-actions github-actions bot force-pushed the internal_types_getters branch from c487428 to 803b845 Compare October 14, 2020 19:36
@andrewjdawson2016 andrewjdawson2016 merged commit 273159d into master Oct 14, 2020
@andrewjdawson2016 andrewjdawson2016 deleted the internal_types_getters branch October 14, 2020 20:01
github-actions bot pushed a commit to vytautas-karpavicius/cadence that referenced this pull request Feb 4, 2021
yux0 pushed a commit to yux0/cadence that referenced this pull request May 4, 2021
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.

3 participants