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

Clarify optionality of databricks_mws_workspaces.deployment_name in docs and examples #1315

Merged
merged 4 commits into from
May 18, 2022

Conversation

easel
Copy link
Contributor

@easel easel commented May 17, 2022

Fixes #1314

@easel easel changed the title Drop deployment_name from aws-workspaces worked example Reflect optionality of deployment_name in docs and examples May 17, 2022
docs/resources/mws_workspaces.md Outdated Show resolved Hide resolved
@nfx nfx changed the title Reflect optionality of deployment_name in docs and examples Clarify optionality of deployment_name attribute of databricks_mws_workspaces in docs and examples May 17, 2022
@nfx nfx changed the title Clarify optionality of deployment_name attribute of databricks_mws_workspaces in docs and examples Clarify optionality of databricks_mws_workspaces.deployment_name in docs and examples May 17, 2022
docs/guides/aws-private-link-workspace.md Outdated Show resolved Hide resolved
docs/guides/aws-private-link-workspace.md Outdated Show resolved Hide resolved
docs/guides/aws-workspace.md Outdated Show resolved Hide resolved
docs/guides/aws-workspace.md Outdated Show resolved Hide resolved
docs/resources/mws_private_access_settings.md Outdated Show resolved Hide resolved
docs/resources/mws_vpc_endpoint.md Outdated Show resolved Hide resolved
docs/resources/mws_workspaces.md Outdated Show resolved Hide resolved
docs/resources/mws_workspaces.md Outdated Show resolved Hide resolved
docs/resources/mws_workspaces.md Outdated Show resolved Hide resolved
docs/resources/mws_workspaces.md Outdated Show resolved Hide resolved
@codecov
Copy link

codecov bot commented May 18, 2022

Codecov Report

Merging #1315 (e5e8485) into master (b28d0ff) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1315   +/-   ##
=======================================
  Coverage   90.60%   90.60%           
=======================================
  Files         118      118           
  Lines       10001    10001           
=======================================
  Hits         9061     9061           
  Misses        559      559           
  Partials      381      381           

@nfx nfx merged commit 8096cc8 into databricks:master May 18, 2022
@easel easel deleted the ignore-deployment-name branch May 18, 2022 19:19
@easel
Copy link
Contributor Author

easel commented May 18, 2022

Thanks for the merge!

@nfx nfx mentioned this pull request May 20, 2022
michael-berk pushed a commit to michael-berk/terraform-provider-databricks that referenced this pull request Feb 15, 2023
…in docs and examples (databricks#1315)

* Drop deployment_name from aws-workspaces worked example
* Reflect optionality in references to deployment_name
* Apply suggestions from code review

Co-authored-by: Serge Smertin <259697+nfx@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants