Skip to content

Commit 81d1346

Browse files
authored
Merge pull request #1170 from sgibson91/update-error-message
Update BinderHub error message for GitHub main/master branches
2 parents 87e78af + d6d3e22 commit 81d1346

File tree

1 file changed

+15
-1
lines changed

1 file changed

+15
-1
lines changed

binderhub/builder.py

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -250,8 +250,22 @@ async def get(self, provider_prefix, _unescaped_spec):
250250
except Exception as e:
251251
await self.fail("Error resolving ref for %s: %s" % (key, e))
252252
return
253+
253254
if ref is None:
254-
await self.fail("Could not resolve ref for %s. Double check your URL." % key)
255+
error_message = ["Could not resolve ref for %s. Double check your URL." % key]
256+
257+
if provider.name == "GitHub":
258+
error_message.append('GitHub recently changed default branches from "master" to "main".')
259+
260+
if provider.unresolved_ref == "master":
261+
error_message.append('Did you mean the "main" branch?')
262+
elif provider.unresolved_ref == "main":
263+
error_message.append('Did you mean the "master" branch?')
264+
265+
else:
266+
error_message.append("Is your repo public?")
267+
268+
await self.fail(" ".join(error_message))
255269
return
256270

257271
self.ref_url = await provider.get_resolved_ref_url()

0 commit comments

Comments
 (0)