New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
A j footer source repo #13
Conversation
ead5027
to
d4f068c
Compare
d4f068c
to
703d393
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! Just one small nitpick (that you would have had no way of knowing, probably, but let's get this fixed before we merge it.)
src/main/java/edu/ucsb/cs156/courses/services/SystemInfoServiceImpl.java
Outdated
Show resolved
Hide resolved
src/test/java/edu/ucsb/cs156/courses/controllers/SystemInfoControllerTests.java
Show resolved
Hide resolved
src/test/java/edu/ucsb/cs156/courses/controllers/SystemInfoControllerTests.java
Show resolved
Hide resolved
src/test/java/edu/ucsb/cs156/courses/controllers/SystemInfoControllerTests.java
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Abel & Junhwan Lee