Skip to content

Conversation

@DKLoehr
Copy link
Contributor

@DKLoehr DKLoehr commented Jul 21, 2025

The CI is complaining about unexpected indentation.

@llvmbot llvmbot added clang Clang issues not falling into any other category clang:modules C++20 modules and Clang Header Modules labels Jul 21, 2025
@llvmbot
Copy link
Member

llvmbot commented Jul 21, 2025

@llvm/pr-subscribers-clang-modules

@llvm/pr-subscribers-clang

Author: Devon Loehr (DKLoehr)

Changes

The CI is complaining about unexpected indentation.


Full diff: https://github.com/llvm/llvm-project/pull/149901.diff

1 Files Affected:

  • (modified) clang/docs/StandardCPlusPlusModules.rst (+2-2)
diff --git a/clang/docs/StandardCPlusPlusModules.rst b/clang/docs/StandardCPlusPlusModules.rst
index fdd350857a3f7..31d0a5e769378 100644
--- a/clang/docs/StandardCPlusPlusModules.rst
+++ b/clang/docs/StandardCPlusPlusModules.rst
@@ -690,9 +690,9 @@ ensure it is reachable, e.g. ``using N::g;``.
 As of Clang 22.x, the Reduced BMI is enabled by default. You may still want to
 use Full BMI with ``-fno-modules-reduced-bmi`` in the following case:
 1. Your build system uses two-phase compilation but it haven't adjusted the
-   implementation for reduced BMI.
+implementation for reduced BMI.
 2. You meet a regression with Reduced BMI that you cannot work around. Please
-   report an issue for this case.
+report an issue for this case.
 
 Experimental Non-Cascading Changes
 ----------------------------------

@DKLoehr DKLoehr requested a review from ChuanqiXu9 July 21, 2025 21:12
@DKLoehr DKLoehr merged commit dcffa3d into llvm:main Jul 21, 2025
13 checks passed
mahesh-attarde pushed a commit to mahesh-attarde/llvm-project that referenced this pull request Jul 28, 2025
The CI is complaining about unexpected indentation. It seems multiple-line list entries must start at the beginning of each line.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clang:modules C++20 modules and Clang Header Modules clang Clang issues not falling into any other category

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants