Add pip_repository documentation

The macros are leaky and otherwise you have to read sources to find out about the kwargs

Fixes #384

Author: alexeagle

.gitattributes

@@ -0,0 +1 @@
+docs/*.md linguist-generated=true

docs/BUILD

@@ -24,6 +24,7 @@ licenses(["notice"])  # Apache 2.0
 _DOCS = {
     "packaging": "//docs:packaging-docs",
     "pip": "//docs:pip-docs",
+    "pip_repository": "//docs:pip-repository",
     "python": "//docs:core-docs",
 }
 
@@ -104,6 +105,18 @@ stardoc(
     ],
 )
 
+stardoc(
+    name = "pip-repository",
+    out = "pip_repository.md_",
+    input = "//python/pip_install:pip_repository.bzl",
+    target_compatible_with = _NOT_WINDOWS,
+    deps = [
+        ":bazel_repo_tools",
+        ":pip_install_bzl",
+        "//third_party/github.com/bazelbuild/bazel-skylib/lib:versions",
+    ],
+)
+
 stardoc(
     name = "packaging-docs",
     out = "packaging.md_",

docs/pip.md

@@ -25,7 +25,8 @@ def pip_install(requirements = None, name = "pip", **kwargs):
 
     Those dependencies become available in a generated `requirements.bzl` file.
 
-    This macro runs a repository rule that invokes `pip`. In your WORKSPACE file:
+    This macro wraps the [`pip_repository`](./pip_repository.md) rule that invokes `pip`.
+    In your WORKSPACE file:
 
     ```python
     pip_install(
@@ -88,7 +89,7 @@ def pip_install(requirements = None, name = "pip", **kwargs):
     Args:
         requirements (Label): A 'requirements.txt' pip requirements file.
         name (str, optional): A unique name for the created external repository (default 'pip').
-        **kwargs (dict): Keyword arguments passed directly to the `pip_repository` repository rule.
+        **kwargs (dict): Additional arguments to the [`pip_repository`](./pip_repository.md) repository rule.
     """
 
     # Just in case our dependencies weren't already fetched
@@ -107,7 +108,8 @@ def pip_parse(requirements_lock, name = "pip_parsed_deps", **kwargs):
     Those dependencies become available in a generated `requirements.bzl` file.
     You can instead check this `requirements.bzl` file into your repo, see the "vendoring" section below.
 
-    This macro runs a repository rule that invokes `pip`. In your WORKSPACE file:
+    This macro wraps the [`pip_repository`](./pip_repository.md) rule that invokes `pip`, with `incremental` set.
+    In your WORKSPACE file:
 
     ```python
     load("@rules_python//python:pip.bzl", "pip_parse")
@@ -191,10 +193,11 @@ def pip_parse(requirements_lock, name = "pip_parsed_deps", **kwargs):
             of 'requirements' no resolve will take place and pip_repository will create
             individual repositories for each of your dependencies so that wheels are
             fetched/built only for the targets specified by 'build/run/test'.
+            Note that if your lockfile is platform-dependent, you can use the `requirements_[platform]`
+            attributes.
         name (str, optional): The name of the generated repository. The generated repositories
             containing each requirement will be of the form <name>_<requirement-name>.
-        **kwargs (dict): Additional keyword arguments for the underlying
-            `pip_repository` rule.
+        **kwargs (dict): Additional arguments to the [`pip_repository`](./pip_repository.md) repository rule.
     """
 
     # Just in case our dependencies weren't already fetched

docs/pip_repository.md

@@ -126,7 +126,7 @@ def _locked_requirements(rctx):
         requirements_txt = rctx.attr.requirements_darwin
     elif os.startswith("linux") and rctx.attr.requirements_linux != None:
         requirements_txt = rctx.attr.requirements_linux
-    elif "win" in os:
+    elif "win" in os and rctx.attr.requirements_windows != None:
         requirements_txt = rctx.attr.requirements_windows
     if not requirements_txt:
         fail("""\