Output GitHub properties for repos and pull requests

JAORMX · JAORMX · commit 52177410f5fb · 2024-12-17T12:02:52.000+02:00
These were missing and are needed for relying less on hardcoded keys and
moving towards a property-based system.

Signed-off-by: Juan Antonio Osorio &lt;ozz@stacklok.com&gt;
diff --git a/internal/providers/github/properties/pull_request.go b/internal/providers/github/properties/pull_request.go
@@ -212,5 +212,6 @@ func PullRequestV1FromProperties(props *properties.Properties) (*pbinternal.Pull
 		Action:         props.GetProperty(PullPropertyAction).GetString(),
 		BaseCloneUrl:   props.GetProperty(PullPropertyBaseCloneURL).GetString(),
 		TargetCloneUrl: props.GetProperty(PullPropertyTargetCloneURL).GetString(),
+		Properties:     props.ToProtoStruct(),
 	}, nil
 }
diff --git a/internal/providers/github/properties/repository.go b/internal/providers/github/properties/repository.go
@@ -225,6 +225,7 @@ func RepoV1FromProperties(repoProperties *properties.Properties) (*minderv1.Repo
 		IsFork:        isFork,
 		DefaultBranch: repoProperties.GetProperty(RepoPropertyDefaultBranch).GetString(),
 		License:       repoProperties.GetProperty(RepoPropertyLicense).GetString(),
+		Properties:    repoProperties.ToProtoStruct(),
 	}
 
 	return pbRepo, nil

Original file line number	Diff line number	Diff line change
`@@ -212,5 +212,6 @@ func PullRequestV1FromProperties(props properties.Properties) (pbinternal.Pull`
`212`	`212`	`Action: props.GetProperty(PullPropertyAction).GetString(),`
`213`	`213`	`BaseCloneUrl: props.GetProperty(PullPropertyBaseCloneURL).GetString(),`
`214`	`214`	`TargetCloneUrl: props.GetProperty(PullPropertyTargetCloneURL).GetString(),`
	`215`	`+ Properties: props.ToProtoStruct(),`
`215`	`216`	`}, nil`
`216`	`217`	`}`
Original file line number	Diff line number	Diff line change
`@@ -225,6 +225,7 @@ func RepoV1FromProperties(repoProperties properties.Properties) (minderv1.Repo`
`225`	`225`	`IsFork: isFork,`
`226`	`226`	`DefaultBranch: repoProperties.GetProperty(RepoPropertyDefaultBranch).GetString(),`
`227`	`227`	`License: repoProperties.GetProperty(RepoPropertyLicense).GetString(),`
	`228`	`+ Properties: repoProperties.ToProtoStruct(),`
`228`	`229`	`}`
`229`	`230`
`230`	`231`	`return pbRepo, nil`