-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge pull request #64 from EricFillion/ch/a3-add-pr-list
[A3] 📚 Add PR list
- Loading branch information
Showing
1 changed file
with
42 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,42 @@ | ||
| # Assignment 3 - Pull Request / Review List: | ||
| For convenience, we have counted all PRs and reviews by each team member. | ||
| Each teammate reviewed all PRs that they did not create. | ||
|
|
||
| ## Tingzhou Jia (TingzhouJia) - 4 PRs, 19 reviews | ||
| | PR name | | ||
| |-| | ||
| | [#44](https://github.com/EricFillion/CMPE-327/pull/44): Tj/register feature | | ||
| | [#58](https://github.com/EricFillion/CMPE-327/pull/58): :memo: add r2_register failure to report.md | | ||
| | [#61](https://github.com/EricFillion/CMPE-327/pull/61): :memo: add modified templates | | ||
| | [#62](https://github.com/EricFillion/CMPE-327/pull/62): :hammer: add logout test | | ||
|
|
||
| ## Eric Fillion (EricFillion) - 7 PRs, 16 reviews | ||
| | PR name | | ||
| |-| | ||
| | [#40](https://github.com/EricFillion/CMPE-327/pull/40): Added GitHub workflow | | ||
| | [#41](https://github.com/EricFillion/CMPE-327/pull/41): Ef/readme build passing | | ||
| | [#42](https://github.com/EricFillion/CMPE-327/pull/42): Login Page First Test Case | | ||
| | [#43](https://github.com/EricFillion/CMPE-327/pull/43): Added the pull request template | | ||
| | [#53](https://github.com/EricFillion/CMPE-327/pull/53): Ef/r1 login test cases | | ||
| | [#54](https://github.com/EricFillion/CMPE-327/pull/54): Ef/r1 failure report | | ||
| | [#57](https://github.com/EricFillion/CMPE-327/pull/57): 📚 fixed duplication | | ||
|
|
||
| ## Jolene Lammers (jolenelammers) - 7 PRs, 16 reviews | ||
| | PR name | | ||
| |-| | ||
| | [#46](https://github.com/EricFillion/CMPE-327/pull/46): jl/common-autologin | | ||
| | [#47](https://github.com/EricFillion/CMPE-327/pull/47): Jl/login test cases | | ||
| | [#49](https://github.com/EricFillion/CMPE-327/pull/49): jl/a3-r3-homepage-test-cases | | ||
| | [#50](https://github.com/EricFillion/CMPE-327/pull/50): Jl/a3 r2 registration test cases | | ||
| | [#51](https://github.com/EricFillion/CMPE-327/pull/51): jl/a3_failure_report | | ||
| | [#52](https://github.com/EricFillion/CMPE-327/pull/52): Jl/a3 backend test | | ||
| | [#59](https://github.com/EricFillion/CMPE-327/pull/59): :truck: Combine homepage test cases into one file. | | ||
|
|
||
| ## Cooper Harasyn (charasyn) - 5 PRs, 18 reviews | ||
| | PR name | | ||
| |-| | ||
| | [#45](https://github.com/EricFillion/CMPE-327/pull/45): ☂️✅ Add shared auto-login functionality for frontend tests | | ||
| | [#48](https://github.com/EricFillion/CMPE-327/pull/48): ⬆️ Update email-validator package to 1.1.2 to fix warning | | ||
| | [#55](https://github.com/EricFillion/CMPE-327/pull/55): [A3] ✅ Add homepage tests R3.1-R3.5, R3.9-R3.11 | | ||
| | [#56](https://github.com/EricFillion/CMPE-327/pull/56): [A3] ✅ Add 404 page tests | | ||
| | [#63](https://github.com/EricFillion/CMPE-327/pull/63): [A3] 📚 Add failures from R3 and R8 to the failures list | |