Skip to content

Commit

Permalink
Merge branch 'master' of https://github.com/nus-cs3281/2024
Browse files Browse the repository at this point in the history
  • Loading branch information
georgetayqy committed Apr 27, 2024
2 parents 15043ef + bdda86b commit b70e6ec
Show file tree
Hide file tree
Showing 3 changed files with 20 additions and 1 deletion.
14 changes: 14 additions & 0 deletions students/cheehongw/knowledge.md
Original file line number Diff line number Diff line change
Expand Up @@ -16,14 +16,28 @@ In CATcher, I focused on identifying the dependencies that needed to be updated.

Beyond that, I have learnt how to use `npm outdated`, `npm-check` etc to identify outdated dependencies. I also learnt how to use `npm ls` to print out the dependency tree of a given package. This was useful in identifying the dependencies that needed to be updated.

Resources
- https://docs.npmjs.com/cli/v6/commands/npm-outdated
- https://www.npmjs.com/package/npm-check
- https://docs.npmjs.com/cli/v6/commands/npm-ls
-
### ESLint migration

I also assisted with the tslint to eslint migration in CATcher. While initially a PR done by a mentee, I had to stay involved and understand the changes made as well. I had to understand how to configure eslint to work with the project. This was a challenging task, but I am glad that I managed to complete it.

Resources:
- https://eslint.org/docs/user-guide/configuring
- https://github.com/angular-eslint/angular-eslint/blob/main/docs/MIGRATING_FROM_TSLINT.md

### i18next

Beyond work done in CATcher, I also worked on SourceAcademy, where I implemented an i18n framework. i18next is a powerful library that allows for easy translation of text in a project. During my implementation of the i18n framework in SourceAcademy, I referenced several implementations of i18n across various established open source repos such as HospitalRun and FreeCodeCamp for any best practices. From these references, I learned how to structure the i18n files and the various translation resources to make it easy for future translators to add on new translations.

Resources:
- https://www.i18next.com/
- https://react.i18next.com/


## Angular Essentials
I had contributed to CATcher as part of IWM, but I have never really approached the Angular aspects of the project.

Expand Down
4 changes: 3 additions & 1 deletion students/cheehongw/progress.md
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,8 @@
| Week 11 | PR Reviewer | [Add documentation for CATcher's parser #1240](https://github.com/CATcher-org/CATcher/pull/1240) |
| Week 13 | PR Reviewer | [Add login redirect #1256](https://github.com/CATcher-org/CATcher/pull/1256) |
| Reading Week | PR Reviewer & Co-author | [Migrate from TSLint to ESLint #1250](https://github.com/CATcher-org/CATcher/pull/1250) | Had to step in to complete the PR
| Reading Week | PR Author | [Upgrade to Angular 13 #1249](https://github.com/CATcher-org/CATcher/pull/1249) |



### WATcher
Expand All @@ -59,7 +61,7 @@
| Week 6 | PR Reviewer | [Upgrade to Angular 11 #252](https://github.com/CATcher-org/WATcher/pull/252)
| Recess | PR Reviewer | [Fix zone testing import error #269](https://github.com/CATcher-org/WATcher/pull/269)

### SourceAcademy
### External Project - SourceAcademy

| Date | Role | Description | Key Achievements |
| ---- | ---- | ----------- | ---------------- |
Expand Down
3 changes: 3 additions & 0 deletions students/sopa301/progress.md
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,9 @@
| 13 | [#2151] Update Stylelint |
| 13 | [#2151] Update CSS-related Major Dependencies |
| 13 | [#2158] Add More Documentation for Title Component |
| Reading | [#2184] Fix Inconsistent Line Number Colours |
| Reading | [#2151] Update Typescript-related Major Dependencies |
| Reading | [#2001] Extract c-file-type-checkboxes from Summary, Authorship and Zoom |

| Week | PRs Reviewed |
| --- | --- |
Expand Down

0 comments on commit b70e6ec

Please sign in to comment.