Skip to content

Coolest WD Feature :) - #13

Open
douglas-kamseu wants to merge 2 commits into
masterfrom
wdAITEst
Open

douglas-kamseu wants to merge 2 commits into
masterfrom
wdAITEst

Conversation

@douglas-kamseu

Copy link
Copy Markdown
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

Comment thread 01 - Drum Kit/main.js
import sys;
import transformers;
import test;

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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?

Comment thread 112 - Maze Game/script.js
return Math.floor(Math.random() * max);
return Math.floor(Math.random() * max);
}

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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.

2 participants