Adding in the documentation that the package color is loaded

[quark67]

Lines 5073-5076 of siunitx.sty (in my local distribution) contains:

\cs_if_exist:NTF \AddToHook
  { \AddToHook { begindocument / before } }
  { \AtBeginDocument }
    { \RequirePackage { color } }

or lines 339-346 of siunitx-print.dtx here contains:

% Color support is always required; to avoid a potential clash we delay to
% the start of the document.
%    \begin{macrocode}
\cs_if_exist:NTF \AddToHook
  { \AddToHook { begindocument / before } }
  { \AtBeginDocument }
    { \RequirePackage { color } }
%    \end{macrocode}

So color package is always loaded. It can be usefull to say this in the manual in order to avoid coding somethinkg like:

\documentclass{article}
\usepackage{siunitx}
\usepackage{color}

Probably not a problem, but not a clean code.

[josephwright]

I don't see the issue: if you are using color, you should load it.

[quark67]

For example with Ti*kZ, I know that it load xcolor, so I don't write \usepackage{xcolor} in my preamble if it contains \usepackage{tikz} (unless if I need some options from the xcolor package).

But for \usepackage{siunitx} I discovered accidentenly that siunitx loads also color, so if I need the color red (for example \textcolor{red}{text}) I have not to load the package color (unless I need more advanced tools provided by xcolor).

It's not a bug, but a suggestion for inform the user.

For example, at the end of the highlighted sentence (page 21 of the current documentation), add a remark in footnote saying that the package loads the color package: