microsoft / fast

The adaptive interface system for modern web experiences.

https://www.fast.design

Other

9.23k stars 590 forks source link

docs: remove pr title extension advice #6925

Closed zpeterson closed 3 months ago

zpeterson commented 5 months ago

Pull Request

📖 Description

Remove the suggestion that contributors use Refined GitHub or Squashed Merge Message because Squashed Merge Message pointed to GitHub's solution for this issue: New options for controlling the default commit message when merging a pull request.

🎫 Issues

None.

👩‍💻 Reviewer Notes

A maintainer/admin with perms to update the repo settings will need to make the update outlined in New options for controlling the default commit message when merging a pull request.

📑 Test Plan

Test default commit message after making the settings update.

✅ Checklist

N/A -- no code change

General

[ ] I have included a change request file using $ yarn change
[ ] I have added tests for my changes.
[ ] I have tested my changes.
[ ] I have updated the project documentation to reflect my changes.
[ ] I have read the CONTRIBUTING documentation and followed the standards for this project.

Component-specific

[ ] I have added a new component
[ ] I have modified an existing component
[ ] I have updated the definition file
[ ] I have updated the configuration file

janechu commented 3 months ago

Thanks for this! We're doing an overhaul of our docs so this ran into some conflicts, I've included your changes here: https://github.com/microsoft/fast/commit/04d2296d5a81bb0996525d74e6c22b2b58e5cca7