Skip to content

Kristenliu/bench 8074#19

Open
kvo3 wants to merge 8 commits into
mainfrom
kristenliu/BENCH--8074
Open

Kristenliu/bench 8074#19
kvo3 wants to merge 8 commits into
mainfrom
kristenliu/BENCH--8074

Conversation

@kvo3
Copy link
Copy Markdown
Contributor

@kvo3 kvo3 commented May 11, 2026

Related to BENCH-8074.

The following changes were made:

  1. first_hour_on_vwb/creating_a_data_collection.ipynb:

    • Removed code for converting workspace to a data collection due to deprecation of set-property command for terra-type field
  2. first_hour_on_vwb/working_with_bq_resources.ipynb

    • Replaced policy parameter with role parameter in "wb group role grant user" command
  3. first_hour_on_vwb/working_with_groups.ipynb

    • Replaced deprecated list-users command with role in "wb group list-users" command
    • Fixed 3 undefined commandList variable issues due to mismatched usage of wb_command variable
    • Replaced policy parameter with role parameter in "wb group role grant user" command
  4. first_hour_on_vwb/working_with_resources.ipynb

    • Removed code for creating a GCP notebook due to deprecation of function

PDFs and HTML version files added for review

notebook-cleanup Claude skill .md file modified after testing findings

@kvo3 kvo3 requested a review from emmarogge May 11, 2026 23:48
kvo3 and others added 4 commits May 12, 2026 09:40
#20)

The following `notebook-cleanup` Claude skill is used for linting code
and structure, clearing notebook cell outputs and generating HTML
snapshots of outputs when making PRs. HTML snapshots are committed to
the repo in a `notebook-snapshots` directory that mirrors the notebook's
relative path in the repo.

Usage:
- Install skills: cp -r claude/skills/notebook-cleanup/*
~/.claude/skills/ (or symlink)
- Restart Claude Code
- Invoke `/notebook-cleanup`

Related to Jira ticket:
[BENCH-8741](https://verily.atlassian.net/browse/BENCH-8741)

[BENCH-8741]:
https://verily.atlassian.net/browse/BENCH-8741?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
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.

1 participant