Skip to content

chore: remove workspaces#137

Merged
miguelvelezsa merged 10 commits intomainfrom
remove-workspaces
Oct 23, 2025
Merged

chore: remove workspaces#137
miguelvelezsa merged 10 commits intomainfrom
remove-workspaces

Conversation

@miguelvelezsa
Copy link
Contributor

Removed workspaces until we migrate all packages, after that we will enable them again.

@miguelvelezsa miguelvelezsa requested a review from a team as a code owner October 23, 2025 00:08
@miguelvelezsa miguelvelezsa merged commit 7785efe into main Oct 23, 2025
11 checks passed
@miguelvelezsa miguelvelezsa deleted the remove-workspaces branch October 23, 2025 20:14
miguelvelezsa added a commit that referenced this pull request Oct 23, 2025
* feat: remove workspaces configuration

* fix: remove preinstall-deps command from github actions

* fix: add required permissions in run_condtional_lint and run_on_changed

* fix: tsconfig extends path
GautamSharda pushed a commit that referenced this pull request Oct 29, 2025
Fixes #136

I've double-checked the behavior of gax-python, and it has
the check like
`if this_settings.retry and this_settings.retry.retry_codes`

Similar code exists here, but JavaScript empty array [] is
truthy, so it passes to the retryable code path. It should check
if the code length explicitly to be consistent with other
languages.
GautamSharda pushed a commit that referenced this pull request Oct 29, 2025
Fixes #136

I've double-checked the behavior of gax-python, and it has
the check like
`if this_settings.retry and this_settings.retry.retry_codes`

Similar code exists here, but JavaScript empty array [] is
truthy, so it passes to the retryable code path. It should check
if the code length explicitly to be consistent with other
languages.
GautamSharda pushed a commit that referenced this pull request Dec 4, 2025
* chore!: upgrade to Node 18

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Update owlbot.py

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Update .OwlBot.lock.yaml

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* run lint

* add TS configuration file for linting

* Delete .github/workflows/issues-no-repro.yaml

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

* Update package.json

* retry test

* extend timeout

* extend timeout to fix flaky test

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@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

Development

Successfully merging this pull request may close these issues.

2 participants