fix: address code review issues

-   fix black exclude parameter in pyproject.toml to properly exclude ONLY directories
-   reintroduce py38 in codegen/__init__.py
-   remove commented-out lines
-   add plotly/validators/_validators.json to pyproject.toml
-   regenerate code

Author: emilykl

README.md

@@ -55,7 +55,7 @@ See the [Python documentation](https://plotly.com/python/) for more examples.
 
 Built on top of [plotly.js](https://github.com/plotly/plotly.js), `plotly.py` is a high-level, declarative charting library. plotly.js ships with over 30 chart types, including scientific charts, 3D graphs, statistical charts, SVG maps, financial charts, and more.
 
-`plotly.py` is [MIT Licensed](https://github.com/plotly/plotly.py/blob/main/LICENSE.txt). Plotly graphs can be viewed in Jupyter and marimo notebooks, standalone HTML files, or integrated into [Dash applications](https://dash.plotly.com/).
+`plotly.py` is [MIT Licensed](https://github.com/plotly/plotly.py/blob/main/LICENSE.txt). Plotly graphs can be viewed in [Jupyter notebooks](https://jupyter.org), other Python notebook software such as [marimo](https://marimo.io), as standalone HTML files, or integrated into [Dash applications](https://dash.plotly.com/).
 
 [Contact us](https://plotly.com/consulting-and-oem/) for consulting, dashboard development, application integration, and feature additions.
 

codegen/__init__.py

@@ -29,7 +29,7 @@
 
 # Target Python version for code formatting with Black.
 # Must be one of the values listed in pyproject.toml.
-BLACK_TARGET_VERSIONS = "py39 py310 py311 py312"
+BLACK_TARGET_VERSIONS = "py38 py39 py310 py311 py312"
 
 
 # Import notes

codegen/validators.py

@@ -85,7 +85,6 @@ def write_validator_json(outdir, params: dict):
     filepath = opath.join(outdir, "validators", "_validators.json")
     with open(filepath, "w") as f:
         f.write(json.dumps(params, indent=4))
-        # f.write(str(params))
 
 
 def build_data_validator_params(base_trace_node: TraceNode):

plotly/basedatatypes.py

@@ -473,7 +473,6 @@ class is a subclass of both BaseFigure and widgets.DOMWidget.
         DataValidator = ValidatorCache.get_validator("", "data")
         FramesValidator = ValidatorCache.get_validator("", "frames")
         LayoutValidator = ValidatorCache.get_validator("", "layout")
-        # from .validators import DataValidator, LayoutValidator, FramesValidator
 
         super(BaseFigure, self).__init__()
 

pyproject.toml

@@ -63,15 +63,15 @@ plotly = [
     "package_data/*",
     "package_data/templates/*",
     "package_data/datasets/*",
+    "plotly/validators/_validators.json"
 ]
 
 [tool.black]
 line-length = 88
 target_version = ['py38', 'py39', 'py310', 'py311', 'py312']
 include = '\.pyi?$'
 exclude = '''
-
-(
+/(
       \.eggs         # exclude a few common directories in the
     | \.git          # root of the project
     | \.hg
@@ -85,7 +85,7 @@ exclude = '''
     | js
     | submodules
     | plotly/matplotlylib/mplexporter
-)
+)/
 '''
 
 [tool.jupyter-packaging.builder]