Ryan Poon

Results 15 comments of Ryan Poon

We can use the currently implemented [Hash Mode](https://router.vuejs.org/guide/essentials/history-mode#Hash-Mode) or [HTML5 Mode](https://router.vuejs.org/guide/essentials/history-mode#HTML5-Mode) of Vue Router to achieve this. However, Hash Mode in its current implementation seems to be [buggy](https://github.com/reposense/RepoSense/pull/2086), so it...

While the rules have been removed from `stylelint v15` onwards, they have recently been moved to a [community plugin](https://www.npmjs.com/package/@stylistic/stylelint-plugin). If we use this plugin, we can update `stylelint` to the...

As discussed, this is not the [first time](https://github.com/TEAMMATES/teammates/pull/9166) Docker has been introduced to the NUS-OSS ecosystem. Given that RepoSense is an on-premise system, it makes much more sense to introduce...

This looks pretty neat! I'm thinking that for "changing group by", if we highlight a repo under a user, then switching the "group by" can highlight that user under the...

We can either use the `exist` [assertion](https://docs.cypress.io/guides/references/assertions) or we can use [`scrollIntoView`](https://docs.cypress.io/api/commands/scrollIntoView) as you mentioned. I think it would be better to use `scrollIntoView` because it would more closely follow...

I've also looked at #1613 suggesting snapshot testing, but I have a few concerns. For example, there may be differences between systems that make snapshots unreliable. Here's a sample report...

Thanks @asdfghjkxd for consolidating the patterns! Do you have any data on the effect of this refactoring on the performance? I agree that if the performance increase is negligible/insignificant, we'll...

Closed because I don't have the required competencies (as of now) to properly fix the issue. Here are some things I learned in the process. - Updating Node to 18...

It's interesting to note #2102 somehow fixed the bug mentioned above.