Skip to content

fix: typeless bot updates for a fully functioning packaged version#3461

Merged
feywind merged 6 commits intogoogleapis:mainfrom
feywind:typeless-bot-updates
Oct 27, 2022
Merged

fix: typeless bot updates for a fully functioning packaged version#3461
feywind merged 6 commits intogoogleapis:mainfrom
feywind:typeless-bot-updates

Conversation

@feywind
Copy link
Copy Markdown
Contributor

@feywind feywind commented Oct 27, 2022

This updates the typeless sample bot to fix some issues discovered when trying to integrate with OwlBot.

@feywind feywind requested a review from a team October 27, 2022 19:20
@snippet-bot
Copy link
Copy Markdown

snippet-bot Bot commented Oct 27, 2022

Here is the summary of changes.

You are about to add 1 region tag.
You are about to delete 1 region tag.

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

@@ -0,0 +1,25 @@
#!/usr/bin/env node
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Adding a shebang and bin the main differences?

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.

Yeah, and there are some weirdnesses around module naming/paths and ESM, so I went ahead and separated it out with the right name.

@feywind feywind merged commit 8e93229 into googleapis:main Oct 27, 2022
@feywind feywind deleted the typeless-bot-updates branch October 27, 2022 19:43
@release-please release-please Bot mentioned this pull request Oct 27, 2022
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