Skip to content

docs: add requirements section - #14

Closed
malvads wants to merge 1 commit into
mainfrom
test/slopper-report-pr-v2
Closed

docs: add requirements section#14
malvads wants to merge 1 commit into
mainfrom
test/slopper-report-pr-v2

Conversation

@malvads

@malvads malvads commented Jun 7, 2026

Copy link
Copy Markdown
Owner

Test PR for report via PR creation

@malvads

malvads commented Jun 7, 2026

Copy link
Copy Markdown
Owner Author

/slopper report

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request adds a 'Requirements' section to the README.md file specifying compiler, CMake, and Boost versions. The review feedback correctly identifies inconsistencies between these new requirements and the existing documentation and CMakeLists.txt, providing a suggestion to align the GCC and CMake versions.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread README.md
Comment on lines +27 to +29
- C++20 compatible compiler (GCC 11+, Clang 14+, MSVC 2022+)
- CMake 3.20+
- Boost 1.81+ (Beast, Asio, URL)

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

There are inconsistencies between this new Requirements section and the existing Prerequisites section (lines 212-215), as well as CMakeLists.txt:

  1. CMake: CMakeLists.txt specifies cmake_minimum_required(VERSION 3.10) and the prerequisites section lists CMake 3.10+, but here it says CMake 3.20+.
  2. GCC: The prerequisites section lists GCC 12+, but here it says GCC 11+.

Let's align these to ensure consistency across the documentation and build files.

Suggested change
- C++20 compatible compiler (GCC 11+, Clang 14+, MSVC 2022+)
- CMake 3.20+
- Boost 1.81+ (Beast, Asio, URL)
- C++20 compatible compiler (GCC 12+, Clang 14+, MSVC 2022+)
- CMake 3.10+
- Boost 1.81+ (Beast, Asio, URL)

slopper-bot Bot added a commit to Sloppers/community-list that referenced this pull request Jun 7, 2026
@slopper-bot slopper-bot Bot added the slopper/banned Slopper PR trust analysis label label Jun 7, 2026
@slopper-bot

slopper-bot Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

🔴 Slopper — PR Blocked

This PR was automatically closed by Slopper — reported by maintainer @malvads via /slopper report.

This account has been reported to the Slopper global community list. All Slopper installations will flag this account going forward..


Powered by slopper

@slopper-bot slopper-bot Bot closed this Jun 7, 2026
slopper-bot Bot added a commit to Sloppers/community-list that referenced this pull request Jun 7, 2026
@slopper-bot slopper-bot Bot removed the slopper/banned Slopper PR trust analysis label label Jun 7, 2026
@slopper-bot slopper-bot Bot added the slopper/banned Slopper PR trust analysis label label Jun 7, 2026
slopper-bot Bot added a commit to Sloppers/community-list that referenced this pull request Jun 7, 2026
@slopper-bot slopper-bot Bot removed the slopper/banned Slopper PR trust analysis label label Jun 7, 2026
@slopper-bot slopper-bot Bot added the slopper/banned Slopper PR trust analysis label label Jun 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

slopper/banned Slopper PR trust analysis label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant