Skip to content
This repository was archived by the owner on Aug 21, 2025. It is now read-only.

CD-675 AH fix integer alias#232

Merged
ahaessly merged 2 commits into
developfrom
ah_fix_integer_alias
Feb 9, 2024
Merged

CD-675 AH fix integer alias#232
ahaessly merged 2 commits into
developfrom
ah_fix_integer_alias

Conversation

@ahaessly
Copy link
Copy Markdown
Contributor

@ahaessly ahaessly commented Feb 1, 2024

No description provided.

@ahaessly ahaessly changed the title AH fix integer alias CD-675 AH fix integer alias Feb 2, 2024
@KutnerBroadie KutnerBroadie self-requested a review February 5, 2024 15:19
@ahaessly ahaessly force-pushed the ah_fix_integer_alias branch from 63d1e2a to 21eef57 Compare February 5, 2024 16:27
Comment thread src/main/java/bio/terra/cda/app/generators/EntityCountSqlGenerator.java Outdated
Comment thread src/main/java/bio/terra/cda/app/generators/EntityCountSqlGenerator.java Outdated
Comment thread src/main/java/bio/terra/cda/app/models/DataSetInfo.java Outdated
Copy link
Copy Markdown
Contributor

@fthomas641 fthomas641 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.

Copy link
Copy Markdown
Contributor

@tanner-coon-bh tanner-coon-bh left a comment

Choose a reason for hiding this comment

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

Everything looks good to me.

@KutnerBroadie
Copy link
Copy Markdown
Contributor

Rerun tests

@ahaessly ahaessly merged commit f04d22c into develop Feb 9, 2024
@ahaessly ahaessly deleted the ah_fix_integer_alias branch February 9, 2024 19:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants