Skip to content

Fix bug where reviews would always contain your own level rating instead of that of the reviewer's #2039

Fix bug where reviews would always contain your own level rating instead of that of the reviewer's

Fix bug where reviews would always contain your own level rating instead of that of the reviewer's #2039

Triggered via pull request November 15, 2024 22:06
Status Success
Total duration 1m 29s
Artifacts 2

dotnet.yml

on: pull_request
Build, Test, and Upload Builds
1m 20s
Build, Test, and Upload Builds
Fit to window
Zoom out
Zoom in

Annotations

22 warnings
Build, Test, and Upload Builds
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/checkout@v3, actions/setup-dotnet@v3, actions/[email protected]. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
Build, Test, and Upload Builds: Refresh.GameServer/Endpoints/Game/CommentEndpoints.cs#L91
Dereference of a possibly null reference.
Build, Test, and Upload Builds
Converting null literal or possible null value to non-nullable type.
Build, Test, and Upload Builds
Possible null reference assignment.
Build, Test, and Upload Builds
Converting null literal or possible null value to non-nullable type.
Build, Test, and Upload Builds
Possible null reference return.
Build, Test, and Upload Builds: Refresh.GameServer/Types/Assets/GameAssetType.cs#L480
The switch expression does not handle all possible values of its input type (it is not exhaustive). For example, the pattern 'Refresh.GameServer.Types.Assets.GameAssetType.Unknown' is not covered.
Build, Test, and Upload Builds: Refresh.GameServer/Types/UserData/DisallowedUser.cs#L8
Non-nullable property 'Username' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build, Test, and Upload Builds: Refresh.GameServer/Types/Scores/FriendScoresRequest.cs#L9
Non-nullable property 'Usernames' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build, Test, and Upload Builds: Refresh.GameServer/Types/Reviews/SerializedGameReview.cs#L19
Non-nullable property 'Slot' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build, Test, and Upload Builds: Refresh.GameServer/Types/Reviews/SerializedGameReview.cs#L22
Non-nullable property 'Reviewer' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build, Test, and Upload Builds
Converting null literal or possible null value to non-nullable type.
Build, Test, and Upload Builds
Possible null reference assignment.
Build, Test, and Upload Builds
Converting null literal or possible null value to non-nullable type.
Build, Test, and Upload Builds
Possible null reference return.
Build, Test, and Upload Builds: Refresh.GameServer/Types/UserData/DisallowedUser.cs#L8
Non-nullable property 'Username' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build, Test, and Upload Builds: Refresh.GameServer/Endpoints/Game/CommentEndpoints.cs#L91
Dereference of a possibly null reference.
Build, Test, and Upload Builds: Refresh.GameServer/Types/Scores/FriendScoresRequest.cs#L9
Non-nullable property 'Usernames' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build, Test, and Upload Builds: Refresh.GameServer/Types/Reviews/SerializedGameReview.cs#L19
Non-nullable property 'Slot' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build, Test, and Upload Builds: Refresh.GameServer/Types/Reviews/SerializedGameReview.cs#L22
Non-nullable property 'Reviewer' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build, Test, and Upload Builds: Refresh.GameServer/Types/Reviews/GameReviewSlot.cs#L10
Non-nullable property 'SlotType' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Deprecation notice: v1, v2, and v3 of the artifact actions
The following artifacts were uploaded using a version of actions/upload-artifact that is scheduled for deprecation: "Refresh for Linux x64", "Refresh for Windows x64". Please update your workflow to use v4 of the artifact actions. Learn more: https://github.blog/changelog/2024-04-16-deprecation-notice-v3-of-the-artifact-actions/

Artifacts

Produced during runtime
Name Size
Refresh for Linux x64
109 MB
Refresh for Windows x64
105 MB