Consider reversing diff

Open
#20 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
45/100
Issue type
Bug
Clarity
Mostly clear
Activity status
Stale
Tech stack
typescript
Domain
tooling

Research direction

Start by tracing the tool's overwrite-existing-file flow and the path used to display differences between the module template and the existing file. Reproduce the reported case, then verify that the diff labels and orientation correctly show which version would overwrite the other.

Written by the indexing model from the issue text.

Description

bug team-wallet-framework wf-bugs

When the tool detects that it would overwrite an existing file with a version from the module template and you choose to see the differences between the two, the diff appears to indicate the opposite: that you have a version from the module template and you want to apply your version on top of it. Should the diff be reversed?

Dominant language
TypeScript
Stars
7
Forks
3
PR merge metrics
No merged PRs in 30d

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from MetaMask/template-sync

All issues in MetaMask/template-sync

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.