[PR #5119] [MERGED] Make switch more clear #17651

Closed
opened 2025-11-02 14:45:10 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-gitea/gitea/pull/5119
Author: @cristaloleg
Created: 10/19/2018
Status: Merged
Merged: 10/19/2018
Merged by: @lunny

Base: masterHead: gocritic-defaultCaseOrder


📝 Commits (1)

📊 Changes

1 file changed (+2 additions, -3 deletions)

View changed files

📝 models/review.go (+2 -3)

📄 Description

Found by go-critic defaultCaseOrder checker

Log:

models/review.go:41:2 defaultCaseOrder consider to make `default` case as first or as last case

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/go-gitea/gitea/pull/5119 **Author:** [@cristaloleg](https://github.com/cristaloleg) **Created:** 10/19/2018 **Status:** ✅ Merged **Merged:** 10/19/2018 **Merged by:** [@lunny](https://github.com/lunny) **Base:** `master` ← **Head:** `gocritic-defaultCaseOrder` --- ### 📝 Commits (1) - [`1e5474c`](https://github.com/go-gitea/gitea/commit/1e5474ccf142541f5b7a3ae200e93f8a8ae951dc) Make switch more clear ### 📊 Changes **1 file changed** (+2 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `models/review.go` (+2 -3) </details> ### 📄 Description Found by go-critic `defaultCaseOrder` checker Log: ``` models/review.go:41:2 defaultCaseOrder consider to make `default` case as first or as last case ``` --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2025-11-02 14:45:10 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#17651