chore: migrate to owlbot - #62
Conversation
google-cla
Bot
added
the
cla: yes
Codecov Report
@@ Coverage Diff @@
## main #62 +/- ##
=======================================
Coverage ? 84.61%
=======================================
Files ? 3
Lines ? 156
Branches ? 19
=======================================
Hits ? 132
Misses ? 24
Partials ? 0 Continue to review full report at Codecov.
|
SurferJeffAtGoogle
left a comment
SurferJeffAtGoogle
left a comment
There was a problem hiding this comment.
Looks good. Have you tried running the the nodejs post processor with these changes? Not sure how it will behave with an empty version list.
bcoe
left a comment
bcoe
left a comment
There was a problem hiding this comment.
Added a couple notes about the migration progress, post process ran and this is looking good though.
| on: | ||
| push: | ||
| branches: | ||
| - $default-branch |
There was a problem hiding this comment.
@sofisl after we merge this, we can update the templates in synthtool to use $default-branch.
| ## Samples | ||
|
|
||
| Samples are in the [`samples/`](https://github.com/googleapis/nodejs-local-auth/tree/main/samples) directory. Each sample's `README.md` has instructions for running its sample. | ||
| Samples are in the [`samples/`](https://github.com/googleapis/nodejs-local-auth/tree/master/samples) directory. Each sample's `README.md` has instructions for running its sample. |
There was a problem hiding this comment.
this one is slightly more of a pain in the neck, and will need to be fixed when we move everything over.
3 participants
Fixes #61