[PR #46] [MERGED] Code has been restructured and task/todo has been completed #56

New issue

Closed

opened 2026-02-28 01:20:43 +03:00 by kerem · 0 comments

kerem commented

2026-02-28 01:20:43 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/tufanbarisyildirim/gonginx/pull/46
Author: @mofantor
Created: 4/17/2024
Status: ✅ Merged
Merged: 4/19/2024
Merged by: @tufanbarisyildirim

Base: master ← Head: restructured_code

📝 Commits (4)

83f6bc8 new feature: link the parent directive to any directive
8a9351b Refactored code structure.
e7ab277 Add more examples and a user guide
8ab810b make linter happy, tidy-up tests

📊 Changes

38 files changed (+1443 additions, -429 deletions)

View changed files

➕ GUIDE.md (+523 -0)
📝 Makefile (+1 -3)
📝 README.md (+10 -6)
📝 config/block.go (+12 -1)
➕ config/block_test.go (+108 -0)
📝 config/config.go (+37 -4)
📝 config/config_test.go (+1 -56)
📝 config/directive.go (+12 -1)
📝 config/http.go (+13 -1)
📝 config/include.go (+37 -1)
📝 config/include_test.go (+1 -24)
➕ config/location.go (+47 -0)
📝 config/lua_block.go (+12 -1)
📝 config/server.go (+12 -1)
📝 config/statement.go (+5 -1)
📝 config/upstream.go (+13 -1)
📝 config/upstream_server.go (+12 -1)
📝 dumper/block_test.go (+25 -122)
➕ dumper/config_test.go (+58 -0)
📝 dumper/directive_test.go (+4 -2)

...and 18 more files

📄 Description

Add more examples and a user guide
link the parent directive to any directive for easier manipulation
move any context wrapper into their own file (remove from parser)
fixed unknown block or directive error

_{🔄 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/tufanbarisyildirim/gonginx/pull/46 **Author:** [@mofantor](https://github.com/mofantor) **Created:** 4/17/2024 **Status:** ✅ Merged **Merged:** 4/19/2024 **Merged by:** [@tufanbarisyildirim](https://github.com/tufanbarisyildirim) **Base:** `master` ← **Head:** `restructured_code` --- ### 📝 Commits (4) - [`83f6bc8`](https://github.com/tufanbarisyildirim/gonginx/commit/83f6bc8cdf973916b44fb778eb2caaa7eb685636) new feature: link the parent directive to any directive - [`8a9351b`](https://github.com/tufanbarisyildirim/gonginx/commit/8a9351be697c1555ad030730cb699675c9a316d6) Refactored code structure. - [`e7ab277`](https://github.com/tufanbarisyildirim/gonginx/commit/e7ab27773243d78dfd550aff46b7e74865e33320) Add more examples and a user guide - [`8ab810b`](https://github.com/tufanbarisyildirim/gonginx/commit/8ab810bbf82fbe649e11d1909816dc60139819f7) make linter happy, tidy-up tests ### 📊 Changes **38 files changed** (+1443 additions, -429 deletions) <details> <summary>View changed files</summary> ➕ `GUIDE.md` (+523 -0) 📝 `Makefile` (+1 -3) 📝 `README.md` (+10 -6) 📝 `config/block.go` (+12 -1) ➕ `config/block_test.go` (+108 -0) 📝 `config/config.go` (+37 -4) 📝 `config/config_test.go` (+1 -56) 📝 `config/directive.go` (+12 -1) 📝 `config/http.go` (+13 -1) 📝 `config/include.go` (+37 -1) 📝 `config/include_test.go` (+1 -24) ➕ `config/location.go` (+47 -0) 📝 `config/lua_block.go` (+12 -1) 📝 `config/server.go` (+12 -1) 📝 `config/statement.go` (+5 -1) 📝 `config/upstream.go` (+13 -1) 📝 `config/upstream_server.go` (+12 -1) 📝 `dumper/block_test.go` (+25 -122) ➕ `dumper/config_test.go` (+58 -0) 📝 `dumper/directive_test.go` (+4 -2) _...and 18 more files_ </details> ### 📄 Description - Add more examples and a user guide - link the parent directive to any directive for easier manipulation - move any context wrapper into their own file (remove from parser) - fixed unknown block or directive error --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>