[PR #559] [MERGED] [Bug] gnome sort: reverted merge commit adding error #987

Open
opened 2026-01-29 15:16:38 +00:00 by claunia · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/TheAlgorithms/C/pull/559
Author: @kvedala
Created: 7/5/2020
Status: Merged
Merged: 7/8/2020
Merged by: @kvedala

Base: masterHead: fix/gnome_sort


📝 Commits (2)

📊 Changes

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

View changed files

📝 sorting/gnome_sort.c (+4 -3)

📄 Description

Description of Change

Pull-request #513 was incorrectly merged adding to errors in the original implementation. This un-does the pull-merge and restores to working and valid algorithm.

References

Checklist

  • Added description of change
  • Added file name matches File name guidelines
  • Added tests and example, test must pass
  • Relevant documentation/comments is changed or added
  • PR title follows semantic commit guidelines
  • Search previous suggestions before making a new one, as yours may be a duplicate.
  • I acknowledge that all my contributions will be made under the project's license.

Notes:


🔄 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/TheAlgorithms/C/pull/559 **Author:** [@kvedala](https://github.com/kvedala) **Created:** 7/5/2020 **Status:** ✅ Merged **Merged:** 7/8/2020 **Merged by:** [@kvedala](https://github.com/kvedala) **Base:** `master` ← **Head:** `fix/gnome_sort` --- ### 📝 Commits (2) - [`16110cd`](https://github.com/TheAlgorithms/C/commit/16110cd0796cd5251d21b0de2ab43bf4934285fa) Revert "Merge pull request #513 from tania-cmyk/master" - [`645ace6`](https://github.com/TheAlgorithms/C/commit/645ace6cd78d6dd35462376261902d455c380868) formatting source-code for 16110cd0796cd5251d21b0de2ab43bf4934285fa ### 📊 Changes **1 file changed** (+4 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `sorting/gnome_sort.c` (+4 -3) </details> ### 📄 Description #### Description of Change <!-- Thank you for your Pull Request. Please provide a description above and review the requirements below. Contributors guide: https://github.com/TheAlgorithms/C-Plus-Plus/CONTRIBUTING.md --> Pull-request #513 was incorrectly merged adding to errors in the original implementation. This *un-does* the pull-merge and restores to working and valid algorithm. #### References <!-- Add any reference to previous pull-request or issue --> #### Checklist <!-- Remove items that do not apply. For completed items, change [ ] to [x]. --> - [x] Added description of change - [ ] Added file name matches [File name guidelines](https://github.com/TheAlgorithms/C-Plus-Plus/blob/master/CONTRIBUTING.md#New-File-Name-guidelines) - [ ] Added tests and example, test must pass - [ ] Relevant documentation/comments is changed or added - [x] PR title follows semantic [commit guidelines](https://github.com/TheAlgorithms/C-Plus-Plus/blob/master/CONTRIBUTING.md#Commit-Guidelines) - [x] Search previous suggestions before making a new one, as yours may be a duplicate. - [x] I acknowledge that all my contributions will be made under the project's license. Notes: <!-- Please add a one-line description for developers or pull request viewers --> --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
claunia added the pull-request label 2026-01-29 15:16:38 +00:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/C#987