diff --git a/models/forgefed/actor.go b/models/forgefed/actor.go
index 34076e0983..1f54566d41 100644
--- a/models/forgefed/actor.go
+++ b/models/forgefed/actor.go
@@ -61,7 +61,7 @@ func newActorID(validatedURI *url.URL, source string) (ActorID, error) {
 	return result, nil
 }
 
-func NewPersonID(uri string, source string) (PersonID, error) {
+func NewPersonID(uri, source string) (PersonID, error) {
 	// TODO: remove after test
 	//if !validation.IsValidExternalURL(uri) {
 	//	return PersonId{}, fmt.Errorf("uri %s is not a valid external url", uri)
@@ -85,7 +85,7 @@ func NewPersonID(uri string, source string) (PersonID, error) {
 	return personID, nil
 }
 
-func NewRepositoryID(uri string, source string) (RepositoryID, error) {
+func NewRepositoryID(uri, source string) (RepositoryID, error) {
 	if !validation.IsAPIURL(uri) {
 		return RepositoryID{}, fmt.Errorf("uri %s is not a valid repo url on this host %s", uri, setting.AppURL+"api")
 	}
diff --git a/models/forgefed/star.go b/models/forgefed/star.go
index 6131739d2c..7e38b07c2c 100644
--- a/models/forgefed/star.go
+++ b/models/forgefed/star.go
@@ -37,7 +37,7 @@ type Star struct {
 
 // StarNew initializes a Star type activity
 // ToDo: May be used later in creating signed activities
-func StarNew(id ap.ID, ob ap.ID) *Star {
+func StarNew(id, ob ap.ID) *Star {
 	a := ap.ActivityNew(id, StarType, ob)
 	o := Star{Activity: *a, Source: ForgejoSourceType}
 	return &o
diff --git a/modules/validation/validateable.go b/modules/validation/validateable.go
index 0bf16fd72b..312450e666 100644
--- a/modules/validation/validateable.go
+++ b/modules/validation/validateable.go
@@ -21,7 +21,7 @@ func IsValid(v Validateable) (bool, error) {
 	return true, nil
 }
 
-func ValidateNotEmpty(value string, fieldName string) []string {
+func ValidateNotEmpty(value, fieldName string) []string {
 	if value == "" {
 		return []string{fmt.Sprintf("Field %v may not be empty", fieldName)}
 	}