Skip to content
This repository was archived by the owner on Apr 3, 2024. It is now read-only.

chore: fix presubmit failures by adjusting docs and pinning a system-test dependency#1113

Merged
mctavish merged 3 commits intogoogleapis:mainfrom
mctavish:quickfix
Nov 17, 2022
Merged

chore: fix presubmit failures by adjusting docs and pinning a system-test dependency#1113
mctavish merged 3 commits intogoogleapis:mainfrom
mctavish:quickfix

Conversation

@mctavish
Copy link
Copy Markdown
Contributor

@mctavish mctavish commented Nov 17, 2022

ci/kokoro: System test is broken without this change.

Newer versions of typescript prevent express-serve-static-core from building properly, which blocks PRs in this repo from being submitted. Given the fact that the typescript version is largely irrelevant for the sample app, this pinning to a specific version is the simplest way to unblock submission of PRs.

Also includes quick change to make the readme easier to read and to help out linkinator, without which ci/docs will fail.

@mctavish mctavish requested review from a team November 17, 2022 18:29
@product-auto-label product-auto-label bot added size: xs Pull request size is extra small. api: clouddebugger Issues related to the googleapis/cloud-debug-nodejs API. samples Issues that are directly related to samples. labels Nov 17, 2022
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 17, 2022

Codecov Report

Base: 65.79% // Head: 65.84% // Increases project coverage by +0.05% 🎉

Coverage data is based on head (0df9fa6) compared to base (92176b0).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1113      +/-   ##
==========================================
+ Coverage   65.79%   65.84%   +0.05%     
==========================================
  Files          21       19       -2     
  Lines        1751     1742       -9     
  Branches      352      352              
==========================================
- Hits         1152     1147       -5     
+ Misses        512      508       -4     
  Partials       87       87              
Impacted Files Coverage Δ
.prettierrc.js
.mocharc.js

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@mctavish mctavish changed the title chore: pin typescript version in sample to 4.8.4 chore: fix presubmit failures by adjusting docs and pinning a system-test dependency Nov 17, 2022
@mctavish mctavish added the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 17, 2022
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Nov 17, 2022
@product-auto-label product-auto-label bot added size: s Pull request size is small. and removed size: xs Pull request size is extra small. labels Nov 17, 2022
@mctavish mctavish requested a review from jasonborg November 17, 2022 19:02
@mctavish mctavish merged commit e5b7efd into googleapis:main Nov 17, 2022
@mctavish mctavish deleted the quickfix branch November 17, 2022 19:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: clouddebugger Issues related to the googleapis/cloud-debug-nodejs API. samples Issues that are directly related to samples. size: s Pull request size is small.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants