Skip to content

Conversation

@grendello
Copy link
Contributor

Local builds would generate hundreds (nearly 700) of messages similar to these:

Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //$schema
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //$schema
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //$schema
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //$schema
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //$schema
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //$schema
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //author
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //author
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //author
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //author
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //classifications
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //classifications
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //classifications
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //identity
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //author
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //author
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //name
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //classifications
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //classifications
Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //description

Disable the localizer for local builds, run it only on CI.


Pull Request
title and
description
should follow the
commit-messages.md workflow documentation, and in particular should include:

  • Useful description of why the change is necessary.
  • Links to issues fixed
  • Unit tests

Local builds would generate hundreds (nearly 700) of messages
similar to these:

    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //$schema
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //$schema
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //$schema
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //$schema
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //$schema
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //$schema
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //author
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //author
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //author
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //author
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //classifications
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //classifications
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //classifications
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //identity
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //author
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //author
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //name
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //classifications
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: The following element in the template.json will not be included in the localizations because it does not match any of the rules for localizable elements: //classifications
    Microsoft.TemplateEngine.TemplateLocalizer.Core.TemplateStringExtractor: Adding into localizable strings: //description

Disable the localizer for local builds, run it only on CI.
Comment on lines -27 to 30
<ItemGroup Condition=" '$(RunningOnCI)' != 'true' ">
<ItemGroup Condition=" '$(RunningOnCI)' == 'true' ">
<PackageReference Include="Microsoft.TemplateEngine.Authoring.Tasks" Version="$(MicrosoftTemplateEngineAuthoringTasksPackageVersion)" PrivateAssets="all" IsImplicitlyDefined="true" />
</ItemGroup>
Copy link
Member

Choose a reason for hiding this comment

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

I was reviewing what @pjcollins wrote on how this would work:

I think the need is to checkin the files -- maybe we could make $(LocalizeTemplates) blank by default and you set to to true to opt in?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants