Skip to content

TerminalApp.Unit.Tests.manifest gets updated when building #12543

Closed

Description

Windows Terminal version

Commit 09e9915

Windows build number

10.0.19041.1415

Other Software

No response

Steps to reproduce

  1. Checkout the latest source (commit 09e9915 or later).
  2. Build the solution.
  3. Look for any modified files in git status.

Expected Behavior

There shouldn't be any modified files.

Actual Behavior

The TerminalApp.Unit.Tests.manifest file is being updated with the same changes that were made to the WindowsTerminal.manifest file in PR #12462.

Shouldn't those changes have been committed at the same time as the WindowsTerminal manifest? Or if the unit test manifest is generated automatically, then maybe it shouldn't be committed at all.

Or am I the only one that's getting this?

Activity

  1. ghost added
    Needs-TriageIt's a new issue that the core contributor team needs to triage at the next triage meeting
    on Feb 21, 2022
  2. ghost added
    Needs-Tag-FixDoesn't match tag requirements
    on Feb 21, 2022
  3. zadjii-msft commented on Feb 21, 2022

    @zadjii-msft
    Member

    What, no, I always build the unit tests.... 😅

    I think there's at least one open PR that's got a similar update in it snuck in, but if you'd like to submit a PR just for this tiny bit then that's fine.

    Frankly those tests should probably be reworked into the broader localtests or fixed so they don't need to do that anymore, but 🤷

  4. ghost added
    In-PRThis issue has a related PR
    on Feb 21, 2022
  5. removed
    Needs-TriageIt's a new issue that the core contributor team needs to triage at the next triage meeting
    on Feb 22, 2022
  6. ghost closed this as completedin #12549on Feb 22, 2022
  7. ghost removed
    In-PRThis issue has a related PR
    on Feb 22, 2022
  8. ghost added a commit that references this issue on Feb 22, 2022
    4016a9f
  9. ghost added
    Resolution-Fix-CommittedFix is checked in, but it might be 3-4 weeks until a release.
    on Feb 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Issue-BugIt either shouldn't be doing this or needs an investigation.Needs-Tag-FixDoesn't match tag requirementsResolution-Fix-CommittedFix is checked in, but it might be 3-4 weeks until a release.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions