Skip to content

samples: remove old TS conversion script to make way for typeless sample bot#1650

Merged
feywind merged 29 commits intogoogleapis:mainfrom
feywind:typeless-bot
Dec 2, 2022
Merged

samples: remove old TS conversion script to make way for typeless sample bot#1650
feywind merged 29 commits intogoogleapis:mainfrom
feywind:typeless-bot

Conversation

@feywind
Copy link
Collaborator

@feywind feywind commented Oct 27, 2022

This removes the old TS machinery in favour of the new typeless sample bot that is currently under testing.

@feywind feywind requested a review from a team as a code owner October 27, 2022 20:46
@feywind feywind requested a review from a team October 27, 2022 20:46
@snippet-bot
Copy link

snippet-bot bot commented Oct 27, 2022

No region tags are edited in this PR.

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

  • Refresh this comment

@generated-files-bot
Copy link

generated-files-bot bot commented Oct 27, 2022

Warning: This pull request is touching the following templated files:

@product-auto-label product-auto-label bot added size: xl Pull request size is extra large. api: pubsub Issues related to the googleapis/nodejs-pubsub API. samples Issues that are directly related to samples. labels Oct 27, 2022
@feywind feywind added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 27, 2022
@feywind
Copy link
Collaborator Author

feywind commented Oct 27, 2022

This is waiting now for both an OwlBot image update and another run of the docs-test.

googleapis/synthtool#1671

@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 1, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 1, 2022
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 1, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 1, 2022
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 1, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 1, 2022
@product-auto-label product-auto-label bot added size: l Pull request size is large. and removed size: xl Pull request size is extra large. labels Nov 1, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 4, 2022
@product-auto-label product-auto-label bot added size: xl Pull request size is extra large. and removed size: l Pull request size is large. labels Dec 1, 2022
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 1, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 1, 2022
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 1, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 1, 2022
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 1, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 1, 2022
@product-auto-label product-auto-label bot added size: l Pull request size is large. and removed size: xl Pull request size is extra large. labels Dec 1, 2022
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 1, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 1, 2022
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 2, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 2, 2022
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 2, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Dec 2, 2022
@feywind feywind merged commit 33ab717 into googleapis:main Dec 2, 2022
@feywind feywind deleted the typeless-bot branch December 2, 2022 21:47
feywind added a commit to feywind/nodejs-pubsub that referenced this pull request Dec 12, 2022
…ple bot (googleapis#1650)

* samples: remove old TS conversion script to make way for typeless sample bot

* docs: obey incorrect linting about copyright year

* build: revert incorrect ci.yaml change

* chore: fix lint issues

* build: reverse owlboy.py changes for now

* 🦉 Updates from OwlBot post-processor

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

* build: test gross fix for babel issue in owlbot

* build: remove meaningless gitignore

* 🦉 Updates from OwlBot post-processor

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

* build: test the right samples for TS

* chore: bring back tsconfig and update samples readme

* build: put converted typescript samples in the right place for system testing

* chore: update owlbot

* chore: revert owlbot.py changes to the original form

* chore: fix broken samples test for commandFor

* chore: remove unneeded JS in the TS folder

* chore: update owlbot.py with shim to use newest typeless bot

* chore: correctly break out command line parameters for testing

* 🦉 Updates from OwlBot post-processor

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

* build: don't copy non-existent JS samples

* build: update owlbot to typeless 1.1.0 image

* chore: clean up owlbot.py comments

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

api: pubsub Issues related to the googleapis/nodejs-pubsub API. samples Issues that are directly related to samples. size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants