Skip to content

Commit

Permalink
fix(ruby-yoshi): don't remove PR link from changelog (#1479)
Browse files Browse the repository at this point in the history
* fix(ruby-yoshi): don't remove PR link from changelog

* fix failing test case

* fix lint error

Co-authored-by: Jeff Ching <[email protected]>
  • Loading branch information
bajajneha27 and chingor13 authored Jun 24, 2022
1 parent f3993a0 commit 6faaebe
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 5 deletions.
2 changes: 1 addition & 1 deletion __snapshots__/ruby-yoshi.js
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,7 @@ exports['RubyYoshi buildReleasePullRequest returns release PR changes with semve
#### Bug Fixes
* update dependency com.google.cloud:google-cloud-spanner to v1.50.0
* update dependency com.google.cloud:google-cloud-storage to v1.120.0
* update dependency com.google.cloud:google-cloud-storage to v1.120.0 ([#1234](https://github.com/googleapis/ruby-test-repo/issues/1234))
---
This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please).
Expand Down
5 changes: 1 addition & 4 deletions src/strategies/ruby-yoshi.ts
Original file line number Diff line number Diff line change
Expand Up @@ -123,10 +123,7 @@ export class RubyYoshi extends BaseStrategy {
// Remove bolded scope from change lines
.replace(/^\* \*\*[\w-]+:\*\* /gm, '* ')
// Remove PR and commit links from pull request title suffixes
.replace(
/( \(\[(\w+|#\d+)\]\(https:\/\/github\.com\/[^)]*\)\))+\s*$/gm,
''
)
.replace(/(\(\[(\w+)\]\(https:\/\/github\.com\/[^)]*\)\))+\s*$/gm, '')
// Standardize on h4 for change type subheaders
.replace(/^### (Features|Bug Fixes|Documentation)$/gm, '#### $1')
// Collapse 2 or more blank lines
Expand Down

0 comments on commit 6faaebe

Please sign in to comment.