Skip to content

Conversation

@douglas-kamseu
Copy link
Collaborator

πŸ› οΈ Fixes Issue (Number)

πŸ‘¨β€πŸ’» Changes proposed and Brief Description

βœ… Check List (Check all the applicable boxes)

  • My code doesn't break any part of the project
  • This PR does not contain plagiarized content.
  • My Addition/Changes works properly and matches the overall repo pattern.
  • The title of my pull request is a short description of the requested changes.

πŸ“„ Note to reviewers

πŸ“· Screenshots

import sys;
import transformers;
import test;

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we need these imports?


const emojiAddFunction = async () => {

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you change this?

<!DOCTYPE html>
<html lang="en">

<head>
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think we need this section.

<div class="number-fact">Enter a number here to know some interesting info 😎 .</div>
</div>
</section>

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you change this?

return Math.floor(Math.random() * max);
return Math.floor(Math.random() * max);
}

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: Unrelated whitespace change.

print("user:"+user)
print("repository:"+repository)
print("pullRequestId:"+pullRequestId)
print("githubToken:"+githubToken)
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why did you remove this line?

@@ -0,0 +1,88 @@
angular.module('app', []).controller('GameCtrl', ['$scope', '$timeout', function ($scope, $timeout) {
$scope.variables = {};
$scope.correct = 0;
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think you need $scope.timer = 0;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants