Skip to content

Commit e34dfd6

Browse files
authored
Clarified more language in README.
1 parent 7d50768 commit e34dfd6

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

README.md

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ test files. To submit a new test file, see the section on ``Creating a New Test
5555

5656
# Naming Schemes
5757

58-
* **Each test xourse is named after a dtx file and every dtx file has a test xourse.**
58+
* **Each test xourse in the masterTestFolder is named after a dtx file.**
5959
Since all development work must be implemented via dtx file (as per CTAN standards) and most development work
6060
tends to involve only a few dtx files (usually just one or two), it is often easiest to test the new code by testing
6161
everything that is implemented by the newly changed dtx code. This testing bed makes that easy to accomplish by merely
@@ -106,24 +106,24 @@ So just the ''problem'' related files would be in the structure as follows:
106106
- numbering.tex (demo/test for numbering scheme)
107107

108108
# Creating a New Test File
109-
If you have determined a meaningful test case that has no appropriate test file, you can do one of two things;
109+
If you have determined a meaningful test case that has no appropriate test file, you can do the following;
110110
1) You should post the issue on the github issues tab of the example repo (**NOT** the ximeraLatex repo!) with
111111
a detailed explanation of what you need to test - make sure to highlight exactly the things that need to be
112112
tested that are not currently testable using the existing test files. In other words, make sure to include:
113-
a) What is currently not able to be tested using existing test files.
114-
b) What specifically you are trying to test overall (in the case that the part that is `untestable` is
113+
- What is currently not able to be tested using existing test files.
114+
- What specifically you are trying to test overall (in the case that the part that is `untestable` is
115115
only a piece of what you are trying to test overall).
116-
c) Why you are trying to test and/or what you are trying to change about Ximera that needs testing.
116+
- Why you are trying to test and/or what you are trying to change about Ximera that needs testing.
117117
2) Next - if you are able, you should make a branch of the main examples repo, and write your new test file.
118118
Once you have the new test file written (and tested) and think it is ready to be merged, submit a pull request
119119
with an explanation of what and how you are going about testing the thing you are submitting a new file for,
120120
along with a reference to the github issue in the examples repo that you submitted from part 1. This will
121121
allow another developer to quickly and easily review the file for submission and then merge it.
122-
a) As a general rule, a different developer (than the author of the new file) should review/merge
122+
- As a general rule, a different developer (than the author of the new file) should review/merge
123123
a new test file for safety.
124-
b) New test files should be verified to make sure they are necessary, suitably succinct in their testing,
124+
- New test files should be verified to make sure they are necessary, suitably succinct in their testing,
125125
and conform to the various naming and design requirements listed in this readme.
126-
c) Once the merge has been completed, whomever reviewed the test file and merged the new test file into
126+
- Once the merge has been completed, whomever reviewed the test file and merged the new test file into
127127
the master test repo *must* be the one to close the github issue. This will help with quickly referencing
128128
who did what if there are any issues later on or if someone needs to verify something (helps minimize
129129
digging through the commit history).

0 commit comments

Comments
 (0)