Refactor name of URL

Author: kyoshidajp

cmd/diagnose.go

@@ -27,7 +27,7 @@ const FETCH_REPOS_PER_ONCE = 20
 
 type Diagnosis struct {
 	Name      string
-	Url       string
+	URL       string
 	Archived  bool
 	Ignored   bool
 	Diagnosed bool
@@ -54,12 +54,12 @@ func FetchRepositoryParams(libs []types.Library, g MedicalTechnician) []github.F
 
 			fmt.Printf("%s\n", lib.Name)
 
-			githubUrl, err := g.SourceCodeURL(lib)
+			url, err := g.SourceCodeURL(lib)
 			if err != nil {
 				return
 			}
 
-			repo, err := github.ParseGitHubUrl(githubUrl)
+			repo, err := github.ParseGitHubURL(url)
 			if err != nil {
 				params = append(params,
 					github.FetchRepositoryParam{
@@ -116,7 +116,7 @@ func Diagnose(d MedicalTechnician, r io.ReadSeekCloserAt, year int, ignores []st
 				isIgnore := slices.Contains(ignores, r.Name)
 				diagnosis := Diagnosis{
 					Name:      r.Name,
-					Url:       r.Url,
+					URL:       r.URL,
 					Archived:  r.Archived,
 					Ignored:   isIgnore,
 					Diagnosed: true,
@@ -233,11 +233,11 @@ func Report(diagnoses map[string]Diagnosis) error {
 			continue
 		}
 		if diagnosis.Archived {
-			errMessages = append(errMessages, fmt.Sprintf("[error] %s (archived): %s", diagnosis.Name, diagnosis.Url))
+			errMessages = append(errMessages, fmt.Sprintf("[error] %s (archived): %s", diagnosis.Name, diagnosis.URL))
 			errCount += 1
 		}
 		if !diagnosis.IsActive {
-			warnMessages = append(warnMessages, fmt.Sprintf("[warn] %s (not-maintained): %s", diagnosis.Name, diagnosis.Url))
+			warnMessages = append(warnMessages, fmt.Sprintf("[warn] %s (not-maintained): %s", diagnosis.Name, diagnosis.URL))
 			warnCount += 1
 		}
 	}

cmd/diagnose_test.go

@@ -12,47 +12,47 @@ func TestDiagnose(t *testing.T) {
 	expect := map[string]Diagnosis{
 		"faker": {
 			Name:      "faker",
-			Url:       "https://github.com/faker-ruby/faker",
+			URL:       "https://github.com/faker-ruby/faker",
 			Archived:  false,
 			Ignored:   false,
 			Diagnosed: true,
 			IsActive:  true,
 		},
 		"concurrent-ruby": {
 			Name:      "concurrent-ruby",
-			Url:       "https://github.com/ruby-concurrency/concurrent-ruby",
+			URL:       "https://github.com/ruby-concurrency/concurrent-ruby",
 			Archived:  false,
 			Ignored:   false,
 			Diagnosed: true,
 			IsActive:  true,
 		},
 		"i18n": {
 			Name:      "i18n",
-			Url:       "https://github.com/ruby-i18n/i18n",
+			URL:       "https://github.com/ruby-i18n/i18n",
 			Archived:  false,
 			Ignored:   true,
 			Diagnosed: true,
 			IsActive:  true,
 		},
 		"method_source": {
 			Name:      "method_source",
-			Url:       "https://github.com/banister/method_source",
+			URL:       "https://github.com/banister/method_source",
 			Archived:  false,
 			Ignored:   false,
 			Diagnosed: true,
 			IsActive:  true,
 		},
 		"paperclip": {
 			Name:      "paperclip",
-			Url:       "https://github.com/thoughtbot/paperclip",
+			URL:       "https://github.com/thoughtbot/paperclip",
 			Archived:  true,
 			Ignored:   false,
 			Diagnosed: true,
 			IsActive:  false,
 		},
 		"dotenv": {
 			Name:      "dotenv",
-			Url:       "https://github.com/bkeepers/dotenv",
+			URL:       "https://github.com/bkeepers/dotenv",
 			Archived:  false,
 			Ignored:   false,
 			Diagnosed: true,

cmd/github/github.go

@@ -23,7 +23,7 @@ type GitHubRepository struct {
 	Name            string
 	Owner           string
 	Repo            string
-	Url             string
+	URL             string
 	Archived        bool
 	LastCommittedAt time.Time
 }
@@ -82,7 +82,7 @@ func (githuburl GitHubURL) Parse() (string, string, error) {
 	return owner, repo, nil
 }
 
-func ParseGitHubUrl(url string) (GitHubRepository, error) {
+func ParseGitHubURL(url string) (GitHubRepository, error) {
 	githubURL := GitHubURL{
 		URL: url,
 	}
@@ -93,7 +93,7 @@ func ParseGitHubUrl(url string) (GitHubRepository, error) {
 	return GitHubRepository{
 		Owner: owner,
 		Repo:  repo,
-		Url:   url,
+		URL:   url,
 	}, nil
 }
 
@@ -160,7 +160,7 @@ func FetchFromGitHub(params []FetchRepositoryParam) []GitHubRepository {
 		repos = append(repos, GitHubRepository{
 			Repo:            string(nodeRepo.NameWithOwner),
 			Archived:        bool(nodeRepo.IsArchived),
-			Url:             string(nodeRepo.Url),
+			URL:             string(nodeRepo.Url),
 			Name:            string(nodeRepo.Name),
 			LastCommittedAt: time.Time(lastCommit.CommittedDate.Time),
 		})

cmd/github/github_test.go

@@ -47,7 +47,7 @@ func TestGitHubRepository_IsActive(t *testing.T) {
 	}
 }
 
-func TestParseGitHubUrl(t *testing.T) {
+func TestParseGitHubURL(t *testing.T) {
 	tests := []struct {
 		name string
 		url  string
@@ -91,7 +91,7 @@ func TestParseGitHubUrl(t *testing.T) {
 
 	for _, tt := range tests {
 		t.Run(tt.name, func(t *testing.T) {
-			r, _ := ParseGitHubUrl(tt.url)
+			r, _ := ParseGitHubURL(tt.url)
 			expect := expects[tt.name]
 			assert.Equal(t, expect.Owner, r.Owner)
 			assert.Equal(t, expect.Repo, r.Repo)
@@ -123,13 +123,13 @@ func TestFetchFromGitHub(t *testing.T) {
 		{
 			Name:     "rails",
 			Repo:     "rails/rails",
-			Url:      "https://github.com/rails/rails",
+			URL:      "https://github.com/rails/rails",
 			Archived: false,
 		},
 		{
 			Name:     "strong_parameters",
 			Repo:     "rails/strong_parameters",
-			Url:      "https://github.com/rails/strong_parameters",
+			URL:      "https://github.com/rails/strong_parameters",
 			Archived: true,
 		},
 	}