starred/C
1
0
Fork 0
mirror of https://github.com/TheAlgorithms/C.git synced 2026-02-17 22:01:45 +00:00
Code Issues 1.4k Packages Projects Releases Wiki Activity

[PR #581] [MERGED] [enhancement] updated cmake checks #1010

New Issue
Open
opened 2026-01-29 15:16:50 +00:00 by claunia · 0 comments
claunia commented 2026-01-29 15:16:50 +00:00
Owner
Copy Link

📋 Pull Request Information

Original PR: https://github.com/TheAlgorithms/C/pull/581
Author: @kvedala
Created: 7/23/2020
Status: ✅ Merged
Merged: 7/23/2020
Merged by: @kvedala

Base: master ← Head: cmake_update

📝 Commits (4)

  • c9f286f added header file checks
  • fd28a68 cmake cleanup and better use of checks
  • 4d2d256 fixed bug in numerical_methods/CMake
  • 1f58d0a add docs to cmake

📊 Changes

4 files changed (+72 additions, -49 deletions)

View changed files

📝 CMakeLists.txt (+25 -1)
📝 client_server/CMakeLists.txt (+40 -41)
📝 numerical_methods/CMakeLists.txt (+7 -3)
📝 sorting/CMakeLists.txt (+0 -4)

📄 Description

Description of Change

  • added checks for include files
  • added conditional compilation for complex.h
  • conditional compilation and link for client_server folder

References

Checklist

  • Added description of change
  • [ ] Added file name matches File name guidelines No new files added
  • 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/581 **Author:** [@kvedala](https://github.com/kvedala) **Created:** 7/23/2020 **Status:** ✅ Merged **Merged:** 7/23/2020 **Merged by:** [@kvedala](https://github.com/kvedala) **Base:** `master` ← **Head:** `cmake_update` --- ### 📝 Commits (4) - [`c9f286f`](https://github.com/TheAlgorithms/C/commit/c9f286f85ba3a5e3e40176057216b0ce2ef0adbd) added header file checks - [`fd28a68`](https://github.com/TheAlgorithms/C/commit/fd28a68b4a813f560556e3ceb65903144968199a) cmake cleanup and better use of checks - [`4d2d256`](https://github.com/TheAlgorithms/C/commit/4d2d256b20d4c0b3cea11100a437627a12b11a2e) fixed bug in numerical_methods/CMake - [`1f58d0a`](https://github.com/TheAlgorithms/C/commit/1f58d0ae250745fd24be7597cf0240b1ac67b16f) add docs to cmake ### 📊 Changes **4 files changed** (+72 additions, -49 deletions) <details> <summary>View changed files</summary> 📝 `CMakeLists.txt` (+25 -1) 📝 `client_server/CMakeLists.txt` (+40 -41) 📝 `numerical_methods/CMakeLists.txt` (+7 -3) 📝 `sorting/CMakeLists.txt` (+0 -4) </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 --> * added checks for include files * added conditional compilation for `complex.h` * conditional compilation and link for `client_server` folder #### 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)~ No new files added - [x] Added tests and example, test must pass - [x] 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 --> <a href="https://gitpod.io/#https://github.com/TheAlgorithms/C/pull/581"><img src="https://gitpod.io/api/apps/github/pbs/github.com/kvedala/C.git/1f58d0ae250745fd24be7597cf0240b1ac67b16f.svg" /></a> --- <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:50 +00:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
bug
dont-close
enhancement
good first issue
hacktoberfest
hacktoberfest
help wanted
invalid
not-an-algorithm
out of date
Practice coding
pull-request
Mirrored from GitHub Pull Request
 question
Stale
wontfix
No Label pull-request
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
claunia
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/C#1010
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?