Skip to content
This repository has been archived by the owner on Dec 14, 2018. It is now read-only.

Rename _GlobalImport.cshtml files to _ViewImports.cshtml #2596

Closed
wants to merge 1 commit into from

Conversation

dougbu
Copy link
Member

@dougbu dougbu commented May 21, 2015

  • also rename files and directories with "GlobalImport" in name
    • nearly blind but avoid "ViewImportss" in new names
  • public API change: ViewHierarchyUtility.GetGlobalImportLocations() -> GetViewImportsLocations()
    • primary source updates were comments, tests, and implementation details

nit:

  • rename NestedGlobalImports.cs file to NestedViewImportsController.cs, matching class

- also rename files and directories with "GlobalImport" in name
 - nearly blind but avoid "ViewImportss" in new names
- public API change: `ViewHierarchyUtility.GetGlobalImportLocations()` -> `GetViewImportsLocations()`
 - primary source updates were comments, tests, and implementation details

nit:
- rename NestedGlobalImports.cs file to NestedViewImportsController.cs, matching class
@Eilon
Copy link
Member

Eilon commented May 21, 2015

Looks :shipit: to me. Add to Announcements repo when you commit.

@@ -51,7 +51,7 @@ private CodeTree OnCacheMiss(ICacheSetContext cacheSetContext)
var pagePath = cacheSetContext.Key;
var getCodeTree = (Func<IFileInfo, CodeTree>)cacheSetContext.State;

// GetOrAdd is invoked for each _ViewStart that might potentially exist in the path.
// GetOrAdd is invoked for each _ViewImports that might potentially exist in the path.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice catch.

@NTaylorMullen
Copy link
Member

:shipit:

1 similar comment
@ajaybhargavb
Copy link
Contributor

:shipit:

@dougbu
Copy link
Member Author

dougbu commented May 21, 2015

8a70172

@dougbu dougbu closed this May 21, 2015
@dougbu dougbu deleted the dougbu/rename.GlobalImport.2489 branch May 21, 2015 19:29
@dougbu
Copy link
Member Author

dougbu commented May 21, 2015

FYI was issue #2489

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants