Skip to content
This repository was archived by the owner on Oct 22, 2024. It is now read-only.

Fix: Packages ReadMe text updates#398

Merged
gauthierpetetin merged 3 commits intomainfrom
FixPackages-ReadMe
Jan 11, 2023
Merged

Fix: Packages ReadMe text updates#398
gauthierpetetin merged 3 commits intomainfrom
FixPackages-ReadMe

Conversation

@michaelmccallam
Copy link
Copy Markdown
Contributor

@michaelmccallam michaelmccallam commented Jan 11, 2023

  • updated paragraph to be consistent with main repo ReadMe

Explanation

Screenshots/Screencaps

Before

After

Manual Testing Steps

Pre-merge author checklist

  • I've clearly explained:
    • What problem this PR is solving
    • How this problem was solved
    • How reviewers can test my changes
  • Sufficient automated test coverage has been added

Pre-merge reviewer checklist

  • Manual testing (e.g. pull and build branch, run in browser, test code being changed)
  • PR is linked to the appropriate GitHub issue
  • IF this PR fixes a bug in the release milestone, add this PR to the release milestone

If further QA is required (e.g. new feature, complex testing steps, large refactor), add the Extension QA Board label.

In this case, a QA Engineer approval will be be required.

- updated paragraph to be consistent with main repo ReadMe
@michaelmccallam michaelmccallam requested a review from a team January 11, 2023 17:43
Added consistent capitalisation on 'MetaMask Desktop'
@gauthierpetetin gauthierpetetin merged commit 5e6390a into main Jan 11, 2023
@michaelmccallam michaelmccallam deleted the FixPackages-ReadMe branch January 11, 2023 19:12
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.

2 participants