Skip to content

Commit

Permalink
fix(docs): Grammatical errors - missing punctuation, missing preposit…
Browse files Browse the repository at this point in the history
…ion, unclear wording (#53183)
  • Loading branch information
jeremyrayjewell committed Jan 16, 2024
1 parent a8f2be4 commit e0027b1
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions docs/codebase-best-practices.md
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ The `!important` keyword may be used to override values in some cases (e.g. acce

### RTL support

We are striving to support right-to-left (RTL) layout in the codebase for languages that are read in this direction. For this you need be mindful of how to style components. Here are some quick rules of thumb to follow:
We are striving to support right-to-left (RTL) layout in the codebase for languages that are read in this direction. For this, you need to be mindful of how to style components. Here are some quick rules of thumb to follow:

- Don't use `float` properties
- Use Flexbox and Grid layouts instead, as they have RTL support already built-in, and those will be easier to maintain and review.
Expand Down Expand Up @@ -173,7 +173,7 @@ api/

#### Integration Tests

Integration tests test the API as a whole. The tests will require mocking, and should not require fixtures beyond the database seeding data, and a method to authenticate.
Integration tests test the API as a whole. The tests will require mocking and should not require fixtures beyond the database seeding data and a method for authentication.

Typically, each integration test file will be directly related to a route. The integration tests are located in the `api/tests/` directory:

Expand Down

0 comments on commit e0027b1

Please sign in to comment.