Remove resolution of repo names against the Databricks Github account

[shreyas-goenka]

Changes

This functionality is not exercised (and will not be anytime soon). Instead we use a map to have first party aliases for supported templates.

cli/cmd/bundle/init.go

Line 21 in 1e46b9f

var aliasedTemplates = map[string]string{

Tests

Existing tests and manually, bundle init still works.

[mgyucht]

When you say it isn't exercised, how can you be sure of that? This is a public method, right? Or at least it isn't exercised in any part of the CLI? It seems like bundle init does use this now?

[pietern]

The only repository with a bundle template is mlops-stacks and for that one we have a direct mapping (i.e. turn that string into the fully qualified URL). Because the logic being removed here a always resolves to the databricks organization, and we have no other repositories with bundle templates, we can safely remove it.

[shreyas-goenka]

More broadly, we have a check that short circuits any template paths that were not full git URLs. So even if there are multiple bundle templates in the wild, this code is not being exercised

cli/cmd/bundle/init.go

Lines 90 to 97 in 1e46b9f

	if !isRepoUrl(templatePath) {
	if templateDir != "" {
	return errors.New("--template-dir can only be used with a Git repository URL")
	}
	// skip downloading the repo because input arg is not a URL. We assume
	// it's a path on the local file system in that case
	return template.Materialize(ctx, configFile, templatePath, outputDir)
	}