Skip to content

Change reda 4#4

Open
WDRevAI wants to merge 2 commits intomasterfrom
change_reda_4
Open

Change reda 4#4
WDRevAI wants to merge 2 commits intomasterfrom
change_reda_4

Conversation

@WDRevAI
Copy link
Copy Markdown
Owner

@WDRevAI WDRevAI commented May 22, 2024

🛠️ 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

@WDRevAI
Copy link
Copy Markdown
Owner Author

WDRevAI commented May 22, 2024

test this please

2 similar comments
@WDRevAI
Copy link
Copy Markdown
Owner Author

WDRevAI commented May 22, 2024

test this please

@WDRevAI
Copy link
Copy Markdown
Owner Author

WDRevAI commented May 22, 2024

test this please

return y;
}

function randomWord () {
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Unnecessary whitespace change

@WDRevAI
Copy link
Copy Markdown
Owner Author

WDRevAI commented May 22, 2024

test this please

@WDRevAI
Copy link
Copy Markdown
Owner Author

WDRevAI commented May 22, 2024

File:05 - Random Password Generator/main.js
File:06 - Calendar/dycalendar.js
code looks good

@WDRevAI
Copy link
Copy Markdown
Owner Author

WDRevAI commented May 22, 2024

test this please

@@ -40,10 +40,18 @@ const words = ['seat', 'pen', 'broad', 'vapor', 'ocean',


function randomNumber (l) {
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Unnecessary whitespace change

@WDRevAI
Copy link
Copy Markdown
Owner Author

WDRevAI commented May 22, 2024

test this please

2 similar comments
@WDRevAI
Copy link
Copy Markdown
Owner Author

WDRevAI commented May 22, 2024

test this please

@WDRevAI
Copy link
Copy Markdown
Owner Author

WDRevAI commented May 22, 2024

test this please

@WDRevAI
Copy link
Copy Markdown
Owner Author

WDRevAI commented May 22, 2024

File:05 - Random Password Generator/main.js
File:06 - Calendar/dycalendar.js
code looks good

@WDRevAI
Copy link
Copy Markdown
Owner Author

WDRevAI commented May 22, 2024

test this please

@@ -40,10 +40,18 @@ const words = ['seat', 'pen', 'broad', 'vapor', 'ocean',


function randomNumber (l) {
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Unnecessary whitespace change

@@ -148,19 +148,6 @@
div.setAttribute("class", "dycalendar-header");
div.setAttribute("data-option", JSON.stringify(option));

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Please remove this extra line.

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.

1 participant