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

Copy-editing for SQL templates #1474

Merged
merged 1 commit into from
Jun 5, 2024

Conversation

lennartkats-db
Copy link
Contributor

Changes

This applies changes suggested by @juliacrawf-db

@lennartkats-db lennartkats-db enabled auto-merge June 5, 2024 07:21
@codecov-commenter
Copy link

codecov-commenter commented Jun 5, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 53.57%. Comparing base (e22dd8a) to head (565ec82).
Report is 132 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1474      +/-   ##
==========================================
+ Coverage   52.25%   53.57%   +1.31%     
==========================================
  Files         317      351      +34     
  Lines       18004    20271    +2267     
==========================================
+ Hits         9408    10860    +1452     
- Misses       7903     8612     +709     
- Partials      693      799     +106     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@pietern pietern changed the title [Trivial] Copy-editing for SQL templates Copy-editing for SQL templates Jun 5, 2024
@lennartkats-db lennartkats-db added this pull request to the merge queue Jun 5, 2024
Merged via the queue into databricks:main with commit 41678fa Jun 5, 2024
5 checks passed
@lennartkats-db lennartkats-db deleted the template-copy-edits branch June 5, 2024 11:19
pietern added a commit that referenced this pull request Jun 5, 2024
Bundles:

This releases fixes an issue introduced in v0.221.0 where managing jobs with a single-node cluster would fail.

 * Fix SQL schema selection in default-sql template ([#1471](#1471)).
 * Copy-editing for SQL templates ([#1474](#1474)).
 * Upgrade TF provider to 1.47.0 ([#1476](#1476)).

Internal:
 * Use latest version of goreleaser action ([#1477](#1477)).
@pietern pietern mentioned this pull request Jun 5, 2024
github-merge-queue bot pushed a commit that referenced this pull request Jun 5, 2024
Bundles:

This releases fixes an issue introduced in v0.221.0 where managing jobs
with a single-node cluster would fail.

* Fix SQL schema selection in default-sql template
([#1471](#1471)).
* Copy-editing for SQL templates
([#1474](#1474)).
* Upgrade TF provider to 1.47.0
([#1476](#1476)).

Internal:
* Use latest version of goreleaser action
([#1477](#1477)).
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