fzdy1914

fzdy1914

Member Since 3 years ago

National University of Singapore,

Experience Points
5
follower
Lessons Completed
4
follow
Lessons Completed
18
stars
Best Reply Awards
25
repos

160 contributions in the last year

Pinned
⚡ Contribution analysis tool for Git repositories
⚡ :ab::four: Inventory Manager Application
⚡ Contribution analysis tool for Git repositories
⚡ Checkstyle is a development tool to help programmers write Java code that adheres to a coding standard. By default it supports the Google Java Style Guide and Sun Code Conventions, but is highly configurable. It can be invoked with an ANT task and a command line program.
Activity
Jan
19
10 hours ago
pull request

fzdy1914 merge to reposense/RepoSense

fzdy1914
fzdy1914

[#1591] Omit percentile if the list is sorted alphabetically

Fixes https://github.com/reposense/RepoSense/issues/1591

Proposed commit message:

Omit percentile if the list is not sorted by contribution

The percentage is only useful when the list is sorted by contribution.
It can be confusing in other cases. The percentage now is only 
displayed when the list is sorted by contribution.
Jan
18
1 day ago
pull request

fzdy1914 merge to reposense/RepoSense

fzdy1914
fzdy1914

Support frontend hot reload on Linux

Proposed commit message:

The hotReloadFrontend gradle task does not support Linux. Apache
ant's `Os` class does not have an OS family specifically for Linux. The
Mac hot reload feature also does not appear to work.

Let's:
- add hot reload for Linux using gradle's OperatingSystem::isLinux
- use `npm run serveOpen` command for both Mac and Linux

Pending - Will need to test this on Mac and Windows.

pull request

fzdy1914 merge to reposense/RepoSense

fzdy1914
fzdy1914

Support frontend hot reload on Linux

Proposed commit message:

The hotReloadFrontend gradle task does not support Linux. Apache
ant's `Os` class does not have an OS family specifically for Linux.

Let's add a hot reload task for Linux using gradle's 
OperatingSystem::isLinux.

Pending - Will need to test this on Mac and Windows.

fzdy1914
fzdy1914

See our discussion in slack for more information.

pull request

fzdy1914 merge to reposense/RepoSense

fzdy1914
fzdy1914

Support frontend hot reload on Linux

Proposed commit message:

The hotReloadFrontend gradle task does not support Linux. Apache
ant's `Os` class does not have an OS family specifically for Linux.

Let's add a hot reload task for Linux using gradle's 
OperatingSystem::isLinux.

Pending - Will need to test this on Mac and Windows.

pull request

fzdy1914 merge to reposense/RepoSense

fzdy1914
fzdy1914

[#1603] Warn if a config file has an unknown column

Fixes https://github.com/reposense/RepoSense/issues/1603

Proposed commit message:

RepoSense does not show any error if a config file has an unknown column
as it is likely to be a user mistake that could affect the output.

Added a new exception (InvalidHeaderException) to handle this as using
`InvalidCsvException` will not terminate the program.

The check is done within CsvParser::validateHeader.

The original code already handles case insensitive matching of column names.
fzdy1914
fzdy1914

feels like InvalidHeaderException should be a child class of InvalidCSVException, others LGTM

Jan
17
2 days ago
pull request

fzdy1914 merge to reposense/RepoSense

fzdy1914
fzdy1914

[#1543] Remove the "Show ramp chart for this period" feature

Fixes #1543

Summary

Removes the "Show ramp chart for this period" anchor and openSummary method that only seems to be used by this feature. There does not appear to be deeper implementation for this feature beyond the above.

Proposed commit message:

The "Show ramp chart for this period" feature does not seem to be
particularly intuitive, and as discussed in #1543, may not be very
useful since users can do this manually. 

Let's remove this option from the front end and the associated 
`openSummary` method.
Jan
15
4 days ago
started
started time in 4 days ago
Activity icon
fork

fzdy1914 forked shadowsocks/go-shadowsocks2

⚡ Modern Shadowsocks in Go
fzdy1914 Apache License 2.0 Updated
fork time in 4 days ago
push

fzdy1914 push reposense/RepoSense

fzdy1914
fzdy1914

README.md: update status badges (#1614)

Some of the status badges on the README.md file are outdated and/or are no longer working.

Let's update the README.md file with the new status badges and remove the outdated ones.

commit sha: 65052fc2039a5d88bbd023b5b8440af862aa8053

push time in 4 days ago
pull request

fzdy1914 pull request reposense/RepoSense

fzdy1914
fzdy1914

README.md: update status badges

Proposed commit message:

Update README.md status badges

Some of the status badges on the README.md file are outdated
and/or are no longer working.

Let's update the README.md file with the new status badges and
remove the outdated ones.
Activity icon
issue

fzdy1914 issue comment reposense/RepoSense

fzdy1914
fzdy1914

README.md: update status badges

Proposed commit message:

Update README.md status badges

Some of the status badges on the README.md file are outdated
and/or are no longer working.

Let's update the README.md file with the new status badges and
remove the outdated ones.
fzdy1914
fzdy1914

The CI is failing interestingly. Can you have a check? @dcshzj

pull request

fzdy1914 merge to reposense/RepoSense

fzdy1914
fzdy1914

README.md: update status badges

Proposed commit message:

Update README.md status badges

Some of the status badges on the README.md file are outdated
and/or are no longer working.

Let's update the README.md file with the new status badges and
remove the outdated ones.
push

fzdy1914 push dcshzj/RepoSense

fzdy1914
fzdy1914

[#1584] Clicking on commit message causes it to shift slightly (#1612)

Clicking on a commit pushes the message box rightwards, which can be disorienting.

Let's add a solid transparent border to prevent it from pushing the content around when visible.

fzdy1914
fzdy1914

Merge branch 'master' into update-readme

commit sha: 1d35e0482b7702a7892e49d5ad4db06f0d6f0ac2

push time in 4 days ago
push

fzdy1914 push reposense/RepoSense

fzdy1914
fzdy1914

[#1584] Clicking on commit message causes it to shift slightly (#1612)

Clicking on a commit pushes the message box rightwards, which can be disorienting.

Let's add a solid transparent border to prevent it from pushing the content around when visible.

commit sha: 8df9afadce4759bd386e36730eee566bc8cf4a10

push time in 4 days ago
Activity icon
issue

fzdy1914 issue reposense/RepoSense

fzdy1914
fzdy1914

Clicking on commit message causes it to shift slightly

Clicking on commit message in commits panel causes it to shift slightly which is slightly disorientating

Tell us about your environment

  • RepoSense Version:
  • OS and Version:
  • Web Browser and Version (if applicable):

https://dashboard-1581-pr-reposense-reposense.surge.sh/

Please include the steps to reproduce the bug.

  1. Open commits panel
  2. Click on a commit message

What was expected to happen?

Text does not move.

What actually happened? Please include a screenshot of the output.

Text moves slightly to the right. image image

If possible, include the URL to your RepoSense report or log files (if any).

pull request

fzdy1914 pull request reposense/RepoSense

fzdy1914
fzdy1914

[#1584] Clicking on commit message causes it to shift slightly

Fixes #1584

Solution: Add a solid transparent border to prevent it from pushing the content around when visible.

Proposed commit message:

Prevent commit message from shifting rightwards when clicked on.

Clicking on a commit pushes the message box rightwards, which can be
disorienting.

Let's add a solid transparent border to prevent it from pushing the 
content around when visible.
pull request

fzdy1914 merge to reposense/RepoSense

fzdy1914
fzdy1914

[#1584] Clicking on commit message causes it to shift slightly

Fixes #1584

Solution: Add a solid transparent border to prevent it from pushing the content around when visible.

Proposed commit message:

Prevent clicking on commit in commits panel from shifting commit
message to the right.

Add a solid transparent border to prevent it from pushing the content
around when visible.
fzdy1914
fzdy1914

Code-wise LGTM, can you improve your commit message a bit?

Dec
26
3 weeks ago
Dec
23
3 weeks ago
started
started time in 3 weeks ago
Dec
14
1 month ago
pull request

fzdy1914 merge to reposense/RepoSense

fzdy1914
fzdy1914

[#1564] Cypress tests folder is not linted

Closes #1564

Proposed commit message:

Lint script: add files under cypress folder

For the frontend, the current "lint" script does not pick up files 
under the "cypress" folder.

This should be fixed so that any programmatic and stylistic errors 
in these files can be detected and corrected.

Let's add the files under the "cypress" folder to the lint script 
and fix violations by existing code.