Skip to content
Merged
Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
44 commits
Select commit Hold shift + click to select a range
f14703f
Add cloudbuild.yaml
danieljbruce May 6, 2026
248638f
Shorten the script
danieljbruce May 6, 2026
cdb2ad1
remove the timeout
danieljbruce May 6, 2026
372a03b
feat: add new Bigtable feature
danieljbruce May 6, 2026
7caf8a0
Merge branch 'migrate-bigtable-to-gcp-cloud-build' of https://github.…
danieljbruce May 6, 2026
55b0dec
Merge branch 'main' into migrate-bigtable-to-gcp-cloud-build
danieljbruce May 6, 2026
7a3fadd
Merge branch 'main' into migrate-bigtable-to-gcp-cloud-build
danieljbruce May 6, 2026
5f930f9
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] May 6, 2026
f3772ea
Merge branch 'migrate-bigtable-to-gcp-cloud-build' of https://github.…
gcf-owl-bot[bot] May 6, 2026
83f73dd
feat: add new Bigtable feature
danieljbruce May 6, 2026
d7782b2
Merge branch 'migrate-bigtable-to-gcp-cloud-build' of https://github.…
danieljbruce May 6, 2026
6be7893
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] May 6, 2026
1f71bd1
Merge branch 'migrate-bigtable-to-gcp-cloud-build' of https://github.…
gcf-owl-bot[bot] May 6, 2026
e6cc411
Changed to `ubuntu` in the yaml file
danieljbruce May 6, 2026
c836303
Merge branch 'migrate-bigtable-to-gcp-cloud-build' of https://github.…
danieljbruce May 6, 2026
1ab42bc
change the dir to handwritten/bigtable
danieljbruce May 6, 2026
8e6ff1d
Change the pre-system test script
danieljbruce May 7, 2026
051f019
Fix the unique id error
danieljbruce May 7, 2026
20b11db
Add pre system tests to the first step
danieljbruce May 7, 2026
a74e61e
Remove the comment changed to
danieljbruce May 7, 2026
0aae31a
Eliminate the first pack and play error
danieljbruce May 8, 2026
af443d1
Fix `require` error with node dependencies
danieljbruce May 8, 2026
57bbdad
Modify the pack and play tests by adding a header
danieljbruce May 8, 2026
f6d43fe
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] May 8, 2026
8dd7d70
Eliminate unnecessary comments
danieljbruce May 8, 2026
fd18c1a
Merge branch 'migrate-bigtable-to-gcp-cloud-build' of https://github.…
danieljbruce May 8, 2026
bfe5155
Restore the system test script
danieljbruce May 8, 2026
e51a2e8
Increase the timeout to 3 hours for the system tes
danieljbruce May 8, 2026
11f861b
Merge branch 'main' into migrate-bigtable-to-gcp-cloud-build
danieljbruce May 8, 2026
88a19c7
Add dependency comment
danieljbruce May 8, 2026
c6a65a8
Merge branch 'migrate-bigtable-to-gcp-cloud-build' of https://github.…
danieljbruce May 11, 2026
83539cc
Do a conditional diff check
danieljbruce May 11, 2026
9f6587c
check against main removed
danieljbruce May 11, 2026
9916905
Add a comment about the packnplay tests
danieljbruce May 11, 2026
3ce9c3f
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] May 11, 2026
45420f8
pin the node version
danieljbruce May 11, 2026
206070f
Apply suggestions from code review
danieljbruce May 12, 2026
93476ce
Change all node runs to node 24
danieljbruce May 12, 2026
b8599b8
Fix the install script to use cjs
danieljbruce May 12, 2026
34475c4
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] May 12, 2026
9fcae3c
Remove initial step for checking the diff
danieljbruce May 12, 2026
11f52d2
Remove workspace check and presystem-test
danieljbruce May 12, 2026
ae907a5
Merge branch 'migrate-bigtable-to-gcp-cloud-build' of https://github.…
danieljbruce May 12, 2026
f786ec8
Merge branch 'main' of https://github.com/googleapis/google-cloud-nod…
danieljbruce May 12, 2026
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Apply suggestions from code review
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
  • Loading branch information
danieljbruce and gemini-code-assist[bot] authored May 12, 2026
commit 206070fa8ad1a33333bf6dcd8da4054cfcef11ac
2 changes: 1 addition & 1 deletion handwritten/bigtable/cloudbuild.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ steps:
dir: 'handwritten/bigtable'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need this step at all since GCB can configure when to run based on change detection? When this config enabled it should only run if there are changes in this directory in the first place, right?

Image

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

That's a good point. In which case, I think we can also remove waitFor: ['check-changes'] which was added earlier because of this step


# 1. Set up Node.js environment
- name: 'node:18'
- name: 'node:24'
entrypoint: 'bash'
args:
- '-c'
Expand Down
8 changes: 7 additions & 1 deletion handwritten/bigtable/system-test/install.ts
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,13 @@ describe('📦 pack-n-play test', () => {
description: 'JavaScript user can use the library',
// Inject the reference directive at the top of the code
// The reference header is needed so packnplay tests recognize require
ts:
packageDir: process.cwd(),
sample: {
description: 'JavaScript user can use the library',
cjs: readFileSync(
'./system-test/fixtures/sample/src/index.js',
).toString(),
},
'/// <reference types="node" />\n' +
readFileSync('./system-test/fixtures/sample/src/index.js').toString(),
// Still needed so the package actually gets installed in the temp environment
Expand Down
Loading