Skip to content

Commit

Permalink
Use old 'Set' typing method for sphinx
Browse files Browse the repository at this point in the history
Summary:
Our conveyors are failing because of a github error: https://www.internalfb.com/sandcastle/workflow/1103381908710640190

Make this old Set type to make ancient machine settings happy

Reviewed By: mpolson64

Differential Revision: D64980148
  • Loading branch information
mgarrard authored and facebook-github-bot committed Oct 25, 2024
1 parent 9803b25 commit fb73c07
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion scripts/validate_sphinx.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
import os
import pkgutil
import re
from typing import Set


# Paths are relative to top-level Ax directory (which is passed into fxn below)
Expand All @@ -30,7 +31,7 @@

# NOTE: Can't use set[str] here due to internal call site of this module
# on an ancient (<py3.9) python version.
def parse_rst(rst_filename: str) -> set[str]:
def parse_rst(rst_filename: str) -> Set[str]:
"""Extract automodule directives from rst."""
ret = set()
with open(rst_filename, "r") as f:
Expand Down

0 comments on commit fb73c07

Please sign in to comment.