Improve wording of e.g and i.e. instances

doc/development/sql.md

@@ -241,7 +241,7 @@ MergeRequest.where(source_project_id: Project.all.select(:id))
 ```
 
 The _only_ time you should use `pluck` is when you actually need to operate on
-the values in Ruby itself (e.g. write them to a file). In almost all other cases
+the values in Ruby itself (for example, writing them to a file). In almost all other cases
 you should ask yourself "Can I not just use a sub-query?".
 
 In line with our `CodeReuse/ActiveRecord` cop, you should only use forms like

doc/development/testing_guide/end_to_end/dynamic_element_validation.md

@@ -52,7 +52,7 @@ Simply put, a required element is a visible HTML element that appears on a UI co
 
 "Visible" can be defined as
 
-- Not having any CSS preventing its display. E.g.: `display: none` or `width: 0px; height: 0px;`
+- Not having any CSS preventing its display, for example, `display: none` or `width: 0px; height: 0px;`
 - Being able to be interacted with by the user
 
 "UI component" can be defined as

doc/development/testing_guide/end_to_end/page_objects.md

@@ -105,7 +105,7 @@ code but **this is deprecated** in favor of the above method for two reasons:
 
 - Consistency: there is only one way to define an element
 - Separation of concerns: QA uses dedicated `data-qa-*` attributes instead of reusing code
-  or classes used by other components (e.g. `js-*` classes etc.)
+  or classes used by other components (for example, `js-*` classes etc.)
 
 ```ruby
 view 'app/views/my/view.html.haml' do

doc/development/testing_guide/flaky_tests.md

@@ -37,9 +37,9 @@ bin/rspec --tag quarantine
 
 Once a test is in quarantine, there are 3 choices:
 
-- Should the test be fixed (i.e. get rid of its flakiness)?
+- Should the test be fixed (that is, get rid of its flakiness)?
 - Should the test be moved to a lower level of testing?
-- Should the test be removed entirely (e.g. because there's already a
+- Should the test be removed entirely (for example, because there's already a
   lower-level test, or it's duplicating another same-level test, or it's testing
   too much etc.)?
 

doc/development/windows.md

@@ -136,7 +136,7 @@ PowerShell has aliases for all of the following commands so you don't have to le
 - `/` ---> `\` (path separator)
 - `cat` ---> `type`
 - `mv` ---> `move`
-- Redirection works the same (i.e. `>` and `2>&1`)
+- Redirection works the same (for example, `>` and `2>&1`)
 - `.\some.exe` to call a local executable
 - curl is available
 - `..` and `.` are available