Skip to content

feat: Modularization: Terraform changes for schema overrides#1841

Merged
manitgupta merged 3 commits intoGoogleCloudPlatform:mainfrom
manitgupta:tf-schema-overrides
Nov 28, 2024
Merged

feat: Modularization: Terraform changes for schema overrides#1841
manitgupta merged 3 commits intoGoogleCloudPlatform:mainfrom
manitgupta:tf-schema-overrides

Conversation

@manitgupta
Copy link
Copy Markdown
Member

@manitgupta manitgupta commented Sep 5, 2024

This should only be merged once #1840 is merged.

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 53.26%. Comparing base (add3dd7) to head (3aee74f).
Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1841      +/-   ##
============================================
+ Coverage     45.56%   53.26%   +7.69%     
+ Complexity     3721     1408    -2313     
============================================
  Files           847      382     -465     
  Lines         50169    20836   -29333     
  Branches       5273     2101    -3172     
============================================
- Hits          22862    11099   -11763     
+ Misses        25632     9054   -16578     
+ Partials       1675      683     -992     
Components Coverage Δ
spanner-templates 68.35% <ø> (+1.34%) ⬆️
spanner-import-export ∅ <ø> (∅)
spanner-live-forward-migration 75.91% <ø> (ø)
spanner-live-reverse-replication 76.87% <ø> (ø)
spanner-bulk-migration 86.56% <ø> (ø)

see 482 files with indirect coverage changes

Copy link
Copy Markdown
Contributor

@bharadwaj-aditya bharadwaj-aditya left a comment

Choose a reason for hiding this comment

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

LGTM

@manitgupta manitgupta marked this pull request as ready for review October 1, 2024 05:39
@manitgupta manitgupta requested a review from a team as a code owner October 1, 2024 05:39
@manitgupta
Copy link
Copy Markdown
Member Author

#1840 is merged, will wait for the release to happen to merge this since it introduces new parameters.

Copy link
Copy Markdown
Contributor

@bharadwaj-aditya bharadwaj-aditya left a comment

Choose a reason for hiding this comment

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

LGTM

@manitgupta manitgupta merged commit 3455819 into GoogleCloudPlatform:main Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Making existing code better size/L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants