Skip to content

Jzzhu and Sequences Solution#391

Open
DavidMaged171 wants to merge 3 commits intokanak22:mainfrom
DavidMaged171:main
Open

Jzzhu and Sequences Solution#391
DavidMaged171 wants to merge 3 commits intokanak22:mainfrom
DavidMaged171:main

Conversation

@DavidMaged171
Copy link
Copy Markdown
Contributor

@DavidMaged171 DavidMaged171 commented Oct 9, 2021

Related Issue

  • Info about the related issue

Closes: #331

Describe the changes you've made

Give a clear description what modifications you have made

Mention any unusual behaviour of your code (Write NA if not)

Any unusual behaviour of your code

Checklist:

(To mark, write x inside the square brackets like this: [x])

  • My code follows the guidelines of this project.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly whereever it was hard to understand.
  • My changes generate no new warnings.

Additional Info (optional)

Any additional information you want to give

@DavidMaged171
Copy link
Copy Markdown
Contributor Author

Solved Problem Jzzhu and Sequences

Copy link
Copy Markdown
Owner

@kanak22 kanak22 left a comment

Choose a reason for hiding this comment

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

  1. rename main.cpp as Jzzhu and Sequences.cpp
  2. remove all the other files in the folder jzzhu and Sequences

@DavidMaged171
Copy link
Copy Markdown
Contributor Author

Changes applied

Copy link
Copy Markdown
Owner

@kanak22 kanak22 left a comment

Choose a reason for hiding this comment

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

Add the file in the correct folder

@DavidMaged171
Copy link
Copy Markdown
Contributor Author

I'm located the file

Copy link
Copy Markdown
Owner

@kanak22 kanak22 left a comment

Choose a reason for hiding this comment

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

The code doesn't have comments, add them asap!

@DavidMaged171
Copy link
Copy Markdown
Contributor Author

Could you accept my PR,Please.

@kanak22
Copy link
Copy Markdown
Owner

kanak22 commented Oct 31, 2021

It doesn't have the required changes done

@DavidMaged171
Copy link
Copy Markdown
Contributor Author

I added All required changes. accept my PR ,please.

@kanak22
Copy link
Copy Markdown
Owner

kanak22 commented Oct 31, 2021

Still no comments are there 🙁

@DavidMaged171
Copy link
Copy Markdown
Contributor Author

My Solution

I added a comments to a code there is a screenshot from my solution

image

@kanak22
Copy link
Copy Markdown
Owner

kanak22 commented Oct 31, 2021

Screenshot (1031)
None of your change is pushed here

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.

Jzzhu and Sequences-450-B-codeforces

2 participants