Skip to content

Commit 60d94b7

Browse files
committed
Give a hard error if outdated environment variables are set
This avoids confusion where the environment variable works with some versions of docs.rs but fails silently with others.
1 parent 127d2b9 commit 60d94b7

File tree

1 file changed

+20
-0
lines changed

1 file changed

+20
-0
lines changed

src/config.rs

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -66,6 +66,26 @@ pub struct Config {
6666

6767
impl Config {
6868
pub fn from_env() -> Result<Self, Error> {
69+
let old_vars = [
70+
("CRATESFYI_PREFIX", "DOCSRS_PREFIX"),
71+
("CRATESFYI_DATABASE_URL", "DOCSRS_DATABASE_URL"),
72+
("CRATESFYI_GITHUB_ACCESSTOKEN", "DOCSRS_GITHUB_ACCESSTOKEN"),
73+
("CRATESFYI_RUSTWIDE_WORKSPACE", "DOCSRS_RUSTWIDE_WORKSPACE"),
74+
("CRATESFYI_TOOLCHAIN", "DOCSRS_TOOLCHAIN"),
75+
("DOCS_RS_DOCKER", "DOCSRS_DOCKER"),
76+
("DOCS_RS_LOCAL_DOCKER_IMAGE", "DOCSRS_DOCKER_IMAGE"),
77+
("DOCS_RS_BULID_CPU_LIMIT", "DOCSRS_BULID_CPU_LIMIT"),
78+
];
79+
for (old_var, new_var) in old_vars {
80+
if std::env::var(old_var).is_ok() {
81+
failure::bail!(
82+
"env variable {} is no longer accepted; use {} instead",
83+
old_var,
84+
new_var
85+
);
86+
}
87+
}
88+
6989
let prefix: PathBuf = require_env("DOCSRS_PREFIX")?;
7090

7191
Ok(Self {

0 commit comments

Comments
 (0)