Skip to content

Commit

Permalink
Minor change proposals (#331)
Browse files Browse the repository at this point in the history
* Today's minor change proposals:
1. Deprecation warning on select
2. Wording

* Update using_the_cloud.md

Minor identation error on vertex_ai_train.yaml file
  • Loading branch information
fmfsa authored Jan 20, 2025
1 parent 71d2e95 commit 1f0d716
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -305,7 +305,7 @@ help us at all. Therefore, use `Any` only when necessary.
def f(x: None | Iterable[int]):
```

because both `list`, `tuple` and `dict` are iterables and therefore can be covered by one type
because all `list`, `tuple` and `dict` are iterables and therefore can be covered by one type
(in this specific case).

This ends the module on coding style. We again want to emphazize that a good coding style is more about having a
Expand Down
2 changes: 1 addition & 1 deletion s6_the_cloud/using_the_cloud.md
Original file line number Diff line number Diff line change
Expand Up @@ -955,7 +955,7 @@ to inject secrets into our code without having to store them in the code itself.
args:
- '-c'
- |
cat config.yaml
cat config.yaml
- name: 'gcr.io/cloud-builders/gcloud'
id: 'Train on vertex AI'
Expand Down

0 comments on commit 1f0d716

Please sign in to comment.