-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge branch 'GoogleCloudPlatform:main' into feat/application-integra…
…tion
- Loading branch information
Showing
131 changed files
with
4,793 additions
and
970 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,36 @@ | ||
Hi there, I'm the Modular magician. I've detected the following information about your changes: | ||
|
||
## Diff report | ||
{{ $diffsLength := len .Diffs }}{{if eq $diffsLength 0 }} | ||
Your PR hasn't generated any diffs, but I'll let you know if a future commit does. | ||
{{else}} | ||
Your PR generated some diffs in downstreams - here they are. | ||
|
||
{{range .Diffs -}} | ||
{{.Title}}: [Diff](https://github.com/modular-magician/{{.Repo}}/compare/auto-pr-{{$.PrNumber}}-old..auto-pr-{{$.PrNumber}}) ({{.DiffStats}}) | ||
{{end -}} | ||
{{end -}} | ||
|
||
{{- $breakingChangesLength := len .BreakingChanges }} | ||
{{- if gt $breakingChangesLength 0}} | ||
## Breaking Change(s) Detected | ||
|
||
The following breaking change(s) were detected within your pull request. | ||
|
||
{{- range .BreakingChanges}} | ||
- {{.}}{{end}} | ||
|
||
If you believe this detection to be incorrect please raise the concern with your reviewer. | ||
If you intend to make this change you will need to wait for a [major release](https://www.terraform.io/plugin/sdkv2/best-practices/versioning#example-major-number-increments) window. | ||
An `override-breaking-change` label can be added to allow merging. | ||
{{end}} | ||
{{.MissingTests}} | ||
{{- $errorsLength := len .Errors}} | ||
{{- if gt $errorsLength 0}} | ||
## Errors | ||
{{range .Errors}} | ||
{{.Title}}: | ||
{{- range .Errors}} | ||
- {{.}}{{end}} | ||
{{end}} | ||
{{- end -}} |
Oops, something went wrong.