Skip to content

feat(typeless-sample-bot): add output path flag, and another pass at module resolution fixing#3498

Merged
feywind merged 4 commits intogoogleapis:mainfrom
feywind:typeless-bot-output-path
Nov 7, 2022
Merged

feat(typeless-sample-bot): add output path flag, and another pass at module resolution fixing#3498
feywind merged 4 commits intogoogleapis:mainfrom
feywind:typeless-bot-output-path

Conversation

@feywind
Copy link
Copy Markdown
Contributor

@feywind feywind commented Nov 7, 2022

This fixes two issues in the typeless sample bot:

  • Adds an outputpath flag to specify where all output samples go. This is needed because synthtool only looks in one directory for template inputs.
  • Another pass at trying to fix Babel's module resolution for the actual sample transformation. All plugins and presets are now being imported directly.

@feywind feywind requested a review from a team November 7, 2022 23:16
@snippet-bot
Copy link
Copy Markdown

snippet-bot Bot commented Nov 7, 2022

Here is the summary of changes.

You are about to add 2 region tags.

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

@product-auto-label product-auto-label Bot added the samples Issues that are directly related to samples. label Nov 7, 2022
@feywind feywind added the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 7, 2022
@gcf-owl-bot gcf-owl-bot Bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 7, 2022
@feywind feywind merged commit 192659d into googleapis:main Nov 7, 2022
@feywind feywind deleted the typeless-bot-output-path branch November 7, 2022 23:32
@release-please release-please Bot mentioned this pull request Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

samples Issues that are directly related to samples.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants