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

chore!: publish gax-nodejs in dual format (CJS and ESM) #1679

Open
wants to merge 59 commits into
base: main
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
59 commits
Select commit Hold shift + click to select a range
d65013f
working compilation
sofisl Nov 21, 2024
287271b
save for now
sofisl Dec 3, 2024
24b6195
save for now
sofisl Dec 5, 2024
3566f28
go back to no esm
sofisl Dec 5, 2024
955896c
check what is going on
sofisl Dec 5, 2024
237abba
try pushing tests up
sofisl Dec 5, 2024
8e57aea
Merge branch 'main' into migrateToEsm
sofisl Dec 5, 2024
a17f484
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Dec 5, 2024
e702960
Merge branch 'migrateToEsm' of https://github.com/googleapis/gax-node…
gcf-owl-bot[bot] Dec 5, 2024
57acad4
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Dec 5, 2024
53d9f3f
Merge branch 'migrateToEsm' of https://github.com/googleapis/gax-node…
gcf-owl-bot[bot] Dec 5, 2024
882a1d9
run 18+ tests
sofisl Dec 6, 2024
1a66338
Merge branch 'migrateToEsm' of github.com:googleapis/gax-nodejs into …
sofisl Dec 6, 2024
1656a87
fix broken error
sofisl Dec 6, 2024
7f68430
rerun lint and fix test in tools
sofisl Dec 6, 2024
200d304
skip test
sofisl Dec 6, 2024
9a34cbd
fix broken linter
sofisl Dec 6, 2024
d27baf9
chore: update post processor
sofisl Dec 6, 2024
c8d0d22
migrate to node 18
sofisl Dec 6, 2024
10837c3
restart tests
sofisl Dec 6, 2024
28ae15b
retry
sofisl Dec 6, 2024
82f62bd
rerun lint
sofisl Dec 6, 2024
82ed56e
fix missing types
sofisl Dec 6, 2024
520e4d8
Update package.json
sofisl Dec 6, 2024
32fd81e
fix system tests
sofisl Dec 6, 2024
2a0dd66
Merge branch 'migrateToEsm' of github.com:googleapis/gax-nodejs into …
sofisl Dec 6, 2024
0f0c1b6
retry
sofisl Dec 6, 2024
be673ee
retry
sofisl Dec 6, 2024
c298e08
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Dec 6, 2024
1aede4a
reruntests
sofisl Dec 7, 2024
4c93886
Merge branch 'migrateToEsm' of github.com:googleapis/gax-nodejs into …
sofisl Dec 7, 2024
f1be309
chore: also export protobuf.js as esm
sofisl Dec 9, 2024
ed5055a
run lint and fix test
sofisl Dec 9, 2024
5c212dc
retry system tests
sofisl Dec 9, 2024
97ab9b9
fix path to test-application
sofisl Dec 9, 2024
901b20f
retry system test
sofisl Dec 10, 2024
fd44396
see difference in browser-test
sofisl Dec 10, 2024
7df63e0
rename assert assertion
sofisl Dec 10, 2024
a98c2d5
fix system test
sofisl Dec 10, 2024
6f306ec
clean up RP
sofisl Dec 10, 2024
14ca692
chore: clean tests
sofisl Dec 10, 2024
98d9366
fix broken test and rerun lint
sofisl Dec 10, 2024
76f5565
fix browser test
sofisl Dec 10, 2024
048e05b
clean up extensions and use node-fetch
sofisl Dec 11, 2024
ccf8a34
run lint
sofisl Dec 11, 2024
896679f
fix node-fetch
sofisl Dec 11, 2024
f4926a7
retry dynamically importing node-fetch
sofisl Dec 11, 2024
2d5e67f
chore: update how we import node-fetch
sofisl Dec 11, 2024
2a2bce1
fix babel to compile dynamic imports correctly
sofisl Dec 11, 2024
00e2472
run lint
sofisl Dec 11, 2024
f07c571
fix path of importing node-fetch dynamically
sofisl Dec 11, 2024
a1b539e
retry using dynamic import
sofisl Dec 12, 2024
fdb98ae
try importing default version of node-fetch
sofisl Dec 12, 2024
f78fcd9
attempt to get browser tests to work with separate browser entrypoint…
sofisl Dec 12, 2024
dc5c20d
get system tests to work
sofisl Dec 12, 2024
7d4f877
add node 22 tests
sofisl Dec 12, 2024
8ebae3c
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] Dec 12, 2024
7c9dbe7
chore: remove `allowSyntheticDefaultImports`
d-goog Dec 13, 2024
a8bca8a
fix: protobuf.js types
d-goog Dec 16, 2024
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
4 changes: 2 additions & 2 deletions .github/.OwlBot.lock.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,5 +13,5 @@
# limitations under the License.
docker:
image: gcr.io/cloud-devrel-public-resources/owlbot-nodejs:latest
digest: sha256:0d39e59663287ae929c1d4ccf8ebf7cef9946826c9b86eda7e85d8d752dbb584
# created: 2024-11-21T22:39:44.342569463Z
digest: sha256:22e41dd7cd82683fa338b647abcc3a29ddb1b17e800b089adc0bec0a3175312c
# created: 2024-10-30T16:51:59.982020867Z
8 changes: 4 additions & 4 deletions .github/sync-repo-settings.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -11,10 +11,10 @@ branchProtectionRules:
- docs
- lint-gax
- lint-tools
- test-gax (14)
- test-gax (16)
- test-tools (14)
- test-tools (16)
- test-gax (18)
- test-gax (20)
- test-tools (18)
- test-tools (20)
- cla/google
- windows-gax
- windows-tools
Expand Down
15 changes: 8 additions & 7 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
node: [14, 16]
node: [18, 20, 22]
steps:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
Expand All @@ -30,7 +30,7 @@ jobs:
runs-on: ubuntu-latest
strategy:
matrix:
node: [14, 16]
node: [18, 20, 22]
steps:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
Expand All @@ -53,7 +53,7 @@ jobs:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: 14
node-version: 18
- run: cd gax; npm install
- run: cd gax; npm test
env:
Expand All @@ -64,7 +64,7 @@ jobs:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: 14
node-version: 18
- run: cd tools; npm install
- run: cd tools; npm test
env:
Expand All @@ -75,7 +75,7 @@ jobs:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: 14
node-version: 18
- run: cd gax; npm install
- run: cd gax; npm run lint
lint-tools:
Expand All @@ -84,7 +84,7 @@ jobs:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: 14
node-version: 18
- run: cd tools; npm install
- run: cd tools; npm run lint
docs:
Expand All @@ -93,9 +93,10 @@ jobs:
- uses: actions/checkout@v4
- uses: actions/setup-node@v4
with:
node-version: 14
node-version: 18
- run: cd gax; npm install
- run: cd gax; npm run docs
- uses: JustinBeckwith/linkinator-action@v1
with:
paths: gax/docs/
config: gax/linkinator.config.json
2 changes: 1 addition & 1 deletion .kokoro/common.cfg

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

24 changes: 0 additions & 24 deletions .kokoro/continuous/node14/common.cfg

This file was deleted.

4 changes: 0 additions & 4 deletions .kokoro/continuous/node14/lint.cfg

This file was deleted.

12 changes: 0 additions & 12 deletions .kokoro/continuous/node14/samples-test.cfg

This file was deleted.

12 changes: 0 additions & 12 deletions .kokoro/continuous/node14/system-test.cfg

This file was deleted.

Empty file.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

24 changes: 0 additions & 24 deletions .kokoro/presubmit/node14/common.cfg

This file was deleted.

12 changes: 0 additions & 12 deletions .kokoro/presubmit/node14/samples-test.cfg

This file was deleted.

12 changes: 0 additions & 12 deletions .kokoro/presubmit/node14/system-test.cfg

This file was deleted.

Empty file.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion .kokoro/release/docs.cfg

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 4 additions & 2 deletions .kokoro/samples-test.sh

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 4 additions & 2 deletions .kokoro/system-test.sh

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion .kokoro/test.bat

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion .kokoro/test.sh

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

20 changes: 20 additions & 0 deletions gax/.babelrc
Original file line number Diff line number Diff line change
@@ -0,0 +1,20 @@
{
"presets": [
"@babel/preset-typescript",
["@babel/preset-env", { "exclude": ["proposal-dynamic-import"] }]
],
"plugins": [
[
"replace-import-extension",
{
"extMapping": {
".js": ".cjs"
}
}
],
"@babel/plugin-syntax-dynamic-import",
"./node_modules/gapic-tools/build/src/replaceESMMockingLib.js",
"./node_modules/gapic-tools/build/src/replaceImportMetaUrl.js",
"./node_modules/gapic-tools/build/src/toggleESMFlagVariable.js"
]
}
6 changes: 6 additions & 0 deletions gax/.eslintignore
Original file line number Diff line number Diff line change
Expand Up @@ -5,3 +5,9 @@ build/
docs/
protos/
samples/generated/
**/*.d.ts
*.d.ts
**/**.d.ts
samples
esm/test/browser-test/karma.conf.js
esm/test/browser-test/webpack.config.js
3 changes: 3 additions & 0 deletions gax/.gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -23,3 +23,6 @@ dist/
*.tgz
**/*.tgz
test/showcase-echo-client/protos/protos.*
**/*.d.ts
*.d.ts
**/**.d.ts
52 changes: 0 additions & 52 deletions gax/.jsdoc.js

This file was deleted.

Loading
Loading