[PR #408] [MERGED] A refactor that can be merge directly to master #515

Closed
opened 2025-11-06 12:01:09 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/veggiemonk/awesome-docker/pull/408
Author: @veggiemonk
Created: 8/28/2017
Status: Merged
Merged: 9/1/2017
Merged by: @veggiemonk

Base: masterHead: refactor/section-bis


📝 Commits (4)

  • 88bbbc7 First step towards section refactoring
  • e551f28 Remove subsection 'Newsletter','Videos Main Account'
  • ba09738 refactor: reorder sections
  • 443d9b8 indent sections, order a little

📊 Changes

2 files changed (+341 additions, -314 deletions)

View changed files

📝 README.md (+338 -311)
📝 index.html (+3 -3)

📄 Description

I made this based on the big amount of work @Moshe-Immerman did. I just reordered a few things, fixed a few links in the TOC and so on.
This can be merged and we can start improving gradually. Most important of all, we can start accepting those PR from other contributors again. Which is the most important I feel.

Unless you have a good reason to not merge this (we might have overlooked something) just let us know and we'll fix it but this PR needs to be merged soon.

Thank you

PS: the reason for this PR is that I cannot push to #406 otherwise I would have and #404 was half way in transition in between commits where I could not understand what was going on... Sorry for another big PR for this refactor.


🔄 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/veggiemonk/awesome-docker/pull/408 **Author:** [@veggiemonk](https://github.com/veggiemonk) **Created:** 8/28/2017 **Status:** ✅ Merged **Merged:** 9/1/2017 **Merged by:** [@veggiemonk](https://github.com/veggiemonk) **Base:** `master` ← **Head:** `refactor/section-bis` --- ### 📝 Commits (4) - [`88bbbc7`](https://github.com/veggiemonk/awesome-docker/commit/88bbbc79eab134a7e10a0c2d3dae6b843f014050) First step towards section refactoring - [`e551f28`](https://github.com/veggiemonk/awesome-docker/commit/e551f28cb0973f7a089a50d775535d90aece7dc1) Remove subsection 'Newsletter','Videos Main Account' - [`ba09738`](https://github.com/veggiemonk/awesome-docker/commit/ba09738ad3911845890977b7a1adc02ddc994974) refactor: reorder sections - [`443d9b8`](https://github.com/veggiemonk/awesome-docker/commit/443d9b801d86645cf4930bc6f5b1f25043c52512) indent sections, order a little ### 📊 Changes **2 files changed** (+341 additions, -314 deletions) <details> <summary>View changed files</summary> 📝 `README.md` (+338 -311) 📝 `index.html` (+3 -3) </details> ### 📄 Description I made this based on the big amount of work @Moshe-Immerman did. I just reordered a few things, fixed a few links in the TOC and so on. This can be merged and we can start improving gradually. Most important of all, we can start accepting those PR from other contributors again. Which is the most important I feel. Unless you have a good reason to not merge this (we might have overlooked something) just let us know and we'll fix it but this PR needs to be merged soon. Thank you PS: the reason for this PR is that I cannot push to #406 otherwise I would have and #404 was half way in transition in between commits where I could not understand what was going on... Sorry for another big PR for this refactor. --- <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-06 12:01:09 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/awesome-docker#515