feat: Add note for Windows users about restarting terminal after setting environment variables

This commit is contained in:
Paul Gauthier (aider) 2024-08-13 09:19:54 -07:00
parent c1fc69ff43
commit d0ab70eba0

View file

@ -3,6 +3,7 @@ import importlib
import json import json
import math import math
import os import os
import platform
import sys import sys
from dataclasses import dataclass, fields from dataclasses import dataclass, fields
from pathlib import Path from pathlib import Path
@ -674,6 +675,10 @@ def sanity_check_model(io, model):
io.tool_error(f"Model {model}: Missing these environment variables:") io.tool_error(f"Model {model}: Missing these environment variables:")
for key in model.missing_keys: for key in model.missing_keys:
io.tool_error(f"- {key}") io.tool_error(f"- {key}")
if platform.system() == "Windows":
io.tool_output("\nNote for Windows users: If you've just set these environment variables using 'setx',")
io.tool_output("you may need to restart your terminal or command prompt for the changes to take effect.")
elif not model.keys_in_environment: elif not model.keys_in_environment:
show = True show = True
io.tool_output(f"Model {model}: Unknown which environment variables are required.") io.tool_output(f"Model {model}: Unknown which environment variables are required.")