Skip to content

Commit

Permalink
Fix ReadtheDocs builds by installing nodejs (#200)
Browse files Browse the repository at this point in the history
* fix: Update imports so that dataclasses is only being imported once in the dm_config_parser.py module
* fix: Update docs to enable building on readthedocs
  • Loading branch information
nfelt14 authored May 3, 2024
1 parent b2670ac commit 6868730
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
1 change: 1 addition & 0 deletions .readthedocs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ build:
os: ubuntu-22.04
tools:
python: '3.11' # This needs to stay in sync with pyproject.toml and any CI scripts
nodejs: '20' # This needs to stay in sync with any CI scripts
python:
install:
- requirements: docs/requirements.txt
Expand Down
3 changes: 1 addition & 2 deletions src/tm_devices/components/dm_config_parser.py
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,6 @@
import os
import pathlib

from dataclasses import dataclass
from types import MappingProxyType
from typing import (
Any,
Expand Down Expand Up @@ -43,7 +42,7 @@


@runtime_checkable
@dataclass
@dataclasses.dataclass
class _DataclassProtocol(Protocol):
"""A Protocol class to allow for type hinting things that accept generic dataclasses."""

Expand Down

0 comments on commit 6868730

Please sign in to comment.