chore: add non-releasing packages to release-please config so their local deps get bumped

[trentm]

This is option 1 from #1917.

Closes: #1917

[codecov]

Codecov Report

Merging #1928 (d5c3334) into main (7895306) will not change coverage.
Report is 1 commits behind head on main.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1928   +/-   ##
=======================================
  Coverage   91.02%   91.02%           
=======================================
  Files         146      146           
  Lines        7478     7478           
  Branches     1497     1497           
=======================================
  Hits         6807     6807           
  Misses        671      671           

[dyladan]

I think you might also need to add it to the manifest file

[trentm]

I think you might also need to add it to the manifest file

My initial naive guess was that with skip-github-release: true for these packages that we would not need to add these to the manifest. That was from @pichlermarc's comments here: #1917 (comment)
and the docs on skip-github-release here: https://github.com/googleapis/release-please/blob/main/docs/manifest-releaser.md#configfile

FWIW, here is a --dry-run run of release-please ... release-pr: https://gist.github.com/trentm/b1a08cdc264b20e054e282d58f719825
I'm not sure from the copious output whether that is a successful test. It didn't blow up on not having the new packages in the manifest at least. However, I ran it against my fork, so it may be of limited value.

[pichlermarc]

Looks good, thanks for taking care of this 👍

[pichlermarc]

I think you might also need to add it to the manifest file

My initial naive guess was that with skip-github-release: true for these packages that we would not need to add these to the manifest. That was from @pichlermarc's comments here: #1917 (comment) and the docs on skip-github-release here: https://github.com/googleapis/release-please/blob/main/docs/manifest-releaser.md#configfile

I also think that should be fine if we don't add these to the manifest file. Let's try without it, adding it later should be an easy fix if it's needed.