prefer slices.Collect

Signed-off-by: Nicolas De Loof <[email protected]>

Author: ndeloof

types/project.go

@@ -121,21 +121,21 @@ func (p *Project) ServicesWithBuild() []string {
 	servicesBuild := p.Services.Filter(func(s ServiceConfig) bool {
 		return s.Build != nil && s.Build.Context != ""
 	})
-	return slices.AppendSeq(make([]string, 0), maps.Keys(servicesBuild))
+	return slices.Collect(maps.Keys(servicesBuild))
 }
 
 func (p *Project) ServicesWithExtends() []string {
 	servicesExtends := p.Services.Filter(func(s ServiceConfig) bool {
 		return s.Extends != nil && *s.Extends != (ExtendsConfig{})
 	})
-	return slices.AppendSeq(make([]string, 0), maps.Keys(servicesExtends))
+	return slices.Collect(maps.Keys(servicesExtends))
 }
 
 func (p *Project) ServicesWithDependsOn() []string {
 	servicesDependsOn := p.Services.Filter(func(s ServiceConfig) bool {
 		return len(s.DependsOn) > 0
 	})
-	return slices.AppendSeq(make([]string, 0), maps.Keys(servicesDependsOn))
+	return slices.Collect(maps.Keys(servicesDependsOn))
 }
 
 func (p *Project) ServicesWithCapabilities() ([]string, []string, []string) {

types/project_test.go

@@ -283,13 +283,13 @@ func TestWithServices(t *testing.T) {
 
 func TestServicesWithBuild(t *testing.T) {
 	p := makeProject()
-	assert.DeepEqual(t, []string{}, p.ServicesWithBuild())
+	assert.Equal(t, len(p.ServicesWithBuild()), 0)
 
 	service, err := p.GetService("service_1")
 	assert.NilError(t, err)
 	service.Build = &BuildConfig{}
 	p.Services["service_1"] = service
-	assert.DeepEqual(t, []string{}, p.ServicesWithBuild())
+	assert.Equal(t, len(p.ServicesWithBuild()), 0)
 
 	service.Build = &BuildConfig{
 		Context: ".",