Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

frigvid

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
KDoc property shouldn't have a colon here, it'll likely render weirdly. It should be <property> <value>.

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
Incorrect creation KDoc property, and date does not follow ISO8601. Additionally, KDoc should have a short description of what it is.

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
This is not resolved and the data class does not have proper indentation.

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
Unnecessary empty starting line in KDoc.

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
Unnecessary empty starting line in KDoc.

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
Dangling KDoc. Additionally, the parameter list and the class's opening bracket should be separated. The class property should also be indented.

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
The if keyword, its parameter list and its opening bracket need to be separated. `if (` and `null) {`.

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
The else keyword and its opening bracket need to be separated. `else {`.

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
It is not about the actual result. It's about the line this comment is placed on. `Result<Unit>` is what the function returns to callers of the function, however, it does not extend or implement it...

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
Unnecessary empty line in KDoc.

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
It is literally the line I'm commenting on. The placement of something being implemented or extended, versus being an expected result is well defined. A class implementing `IProfileEditRepository` ...

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
`FriendRepository` does not expect `IFriendsRepository` as a result. It's implementing an interface and needs to be represented as such following Kotlin's style guide. It should be `) : IF`. Additi...

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
This file has still not been removed, and it's been mentioned multiple times that this is not to be committed.

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
Binaries are _still_ not removed. They've been mentioned multiple times, and are not to be committed.

View on GitHub

frigvid created a review on a pull request on frigvid/mob3000-group11

View on GitHub

frigvid created a review comment on a pull request on frigvid/mob3000-group11
Binaries are _still_ not removed. They've been mentioned multiple times, and are not to be committed.

View on GitHub

Load more