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

chore: Export QueryStruct #2160

Merged
merged 1 commit into from
Oct 31, 2023
Merged

chore: Export QueryStruct #2160

merged 1 commit into from
Oct 31, 2023

Conversation

sfc-gh-jcieslak
Copy link
Collaborator

@sfc-gh-jcieslak sfc-gh-jcieslak commented Oct 30, 2023

This change contains

  • export of QueryStruct
  • rename of its constructor to NewQueryStruct.
    Other name proposal could be to name the type QueryStructBuilder (and other struct builders too) and leave QueryStruct constructor name.

The change enables to create helper functions in definitions file, that are specific to given definition and shouldn't be put into generator common package.

@github-actions
Copy link

Integration tests success for a1c43eea7418255913a1ab06362b0f6b625ead2c

@sfc-gh-jcieslak sfc-gh-jcieslak marked this pull request as ready for review October 30, 2023 10:40
@github-actions
Copy link

Integration tests failure for 2c7704e840ba2ecfffbf9218002b5e21a083f987

1 similar comment
@github-actions
Copy link

Integration tests failure for 2c7704e840ba2ecfffbf9218002b5e21a083f987

@github-actions
Copy link

Integration tests success for 2c7704e840ba2ecfffbf9218002b5e21a083f987

@github-actions
Copy link

Integration tests success for 118571e26d1cfe7d5d9e63c785dce3db5cd0fdd0

@sfc-gh-jcieslak sfc-gh-jcieslak merged commit 5148be4 into main Oct 31, 2023
@sfc-gh-jcieslak sfc-gh-jcieslak deleted the export-query-builder branch October 31, 2023 10:40
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.

2 participants