https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/82402
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/82402
>From 0cb108fa6552b112d5eeb9b82e4f57711fac4a35 Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Tue, 20 Feb 2024 19:44:06 +0100
Subject: [PATCH 1/2] [Docs] Allow building man pages without myst_parser
The pa
https://github.com/AaronBallman approved this pull request.
LGTM aside from the formatting issue @mgorny mentioned.
https://github.com/llvm/llvm-project/pull/82402
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/m
@@ -35,8 +35,16 @@
import sphinx
-if sphinx.version_info >= (3, 0):
-extensions.append("myst_parser")
+# When building man pages, we do not use the markdown pages,
+# So, we can continue without the myst_parser dependencies.
+# Doing so reduces dependencies of some packag
https://github.com/mgorny approved this pull request.
Thank you. From a quick test run with and without `myst-parser`, this seems to
work as intended.
https://github.com/llvm/llvm-project/pull/82402
___
cfe-commits mailing list
cfe-commits@lists.llvm.
https://github.com/mgorny edited https://github.com/llvm/llvm-project/pull/82402
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman commented:
These changes look reasonable to me, but I'd appreciate a final sign off from
@mgorny to make sure this works for his needs.
https://github.com/llvm/llvm-project/pull/82402
___
cfe-commits mailing list
cfe-co
github-actions[bot] wrote:
:warning: Python code formatter, darker found issues in your code. :warning:
You can test this locally with the following command:
``bash
darker --check --diff -r
13e64958a00f9f47fc71027d5b6196906cbc2863...0cb108fa6552b112d5eeb9b82e4f57711fac4a35
clang
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/82402
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: cor3ntin (cor3ntin)
Changes
The pan pages do not depend on the doc present in markdown files, so they can
be built without myst_parser.
Doing so might allow llvm distributions to have less development dependencies.
As we do not have the e
https://github.com/cor3ntin created
https://github.com/llvm/llvm-project/pull/82402
The pan pages do not depend on the doc present in markdown files, so they can
be built without myst_parser.
Doing so might allow llvm distributions to have less development dependencies.
As we do not have the e
11 matches
Mail list logo