Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replaced principals with principles #6739

Merged
merged 1 commit into from
May 22, 2023

Conversation

EddieMachete
Copy link
Contributor

@EddieMachete EddieMachete commented May 19, 2023

Pull Request

📖 Description

Closes #6738

The introduction page has the following line of text:

One of FAST's driving principals is "strive to adopt open, web standards-based approaches as much as possible."

I believe the line is referring to principles as opposed to principals.

As a solution, I have just replaced the word principals with principles.

🎫 Issues

👩‍💻 Reviewer Notes

This is a simple content change. Just make sure the new copy makes sense and the change looks correct.

📑 Test Plan

  • principals has been replaced with principles
Test Case
  1. Open the documentation site and navigate to the introduction page /docs/introduction/
  2. Confirm that the text One of FAST's driving principals is has been changed to One of FAST's driving principles is

✅ Checklist

General

  • I have searched open/closed issues before submitting
  • 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.

⏭ Next Steps

Not applicable

@EddieMachete EddieMachete marked this pull request as ready for review May 19, 2023 23:33
@awentzel awentzel merged commit df53eba into microsoft:master May 22, 2023
KingOfTac pushed a commit that referenced this pull request Jun 17, 2023
janechu pushed a commit that referenced this pull request Jun 10, 2024
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.

Spelling mistake on docs/introduction
4 participants