[PR #75] [MERGED] Refactor process package and introduce ProcessManager{} with tests #15131

Closed
opened 2025-11-02 11:39:33 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/go-gitea/gitea/pull/75
Author: @metalmatze
Created: 11/4/2016
Status: Merged
Merged: 1/17/2017
Merged by: @lunny

Base: masterHead: feature/process


📝 Commits (4)

  • 55ef84b Add a process.Manager singleton with process.GetManager()
  • 76c6b1f Use process.GetManager everywhere
  • 0d94220 Fix godoc comments for process module
  • 7772a39 Increment process counter id after locking the mutex

📊 Changes

10 files changed (+161 additions, -113 deletions)

View changed files

📝 models/git_diff.go (+2 -2)
📝 models/pull.go (+9 -9)
📝 models/release.go (+9 -11)
📝 models/repo.go (+14 -12)
📝 models/repo_editor.go (+2 -2)
📝 models/repo_mirror.go (+2 -2)
📝 models/ssh_key.go (+2 -2)
📝 modules/process/manager.go (+87 -72)
modules/process/manager_test.go (+33 -0)
📝 routers/admin/admin.go (+1 -1)

📄 Description

This is still WIP and other files need to be updated.
Some funcs that use this need get a ProcessManager instance passed.


🔄 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/75 **Author:** [@metalmatze](https://github.com/metalmatze) **Created:** 11/4/2016 **Status:** ✅ Merged **Merged:** 1/17/2017 **Merged by:** [@lunny](https://github.com/lunny) **Base:** `master` ← **Head:** `feature/process` --- ### 📝 Commits (4) - [`55ef84b`](https://github.com/go-gitea/gitea/commit/55ef84b9ac962d6701010fa5c4b18826cb197808) Add a process.Manager singleton with process.GetManager() - [`76c6b1f`](https://github.com/go-gitea/gitea/commit/76c6b1f0b0544171be51c5f02d7b6b5c8afa1ea1) Use process.GetManager everywhere - [`0d94220`](https://github.com/go-gitea/gitea/commit/0d9422047028a28c4fa1edc62ac1fa2705f2119f) Fix godoc comments for process module - [`7772a39`](https://github.com/go-gitea/gitea/commit/7772a3922712d57714447cb4d40dba31be2bfdf0) Increment process counter id after locking the mutex ### 📊 Changes **10 files changed** (+161 additions, -113 deletions) <details> <summary>View changed files</summary> 📝 `models/git_diff.go` (+2 -2) 📝 `models/pull.go` (+9 -9) 📝 `models/release.go` (+9 -11) 📝 `models/repo.go` (+14 -12) 📝 `models/repo_editor.go` (+2 -2) 📝 `models/repo_mirror.go` (+2 -2) 📝 `models/ssh_key.go` (+2 -2) 📝 `modules/process/manager.go` (+87 -72) ➕ `modules/process/manager_test.go` (+33 -0) 📝 `routers/admin/admin.go` (+1 -1) </details> ### 📄 Description This is still WIP and other files need to be updated. Some funcs that use this need get a ProcessManager instance passed. --- <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 11:39:34 -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#15131