Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Adding the Letter R class to the Pattern_Printing class #347

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

WilsoP
Copy link
Contributor

@WilsoP WilsoP commented Oct 8, 2021

Issue Id you have worked upon -

Kindly write your answer here
313

Briefly explain your program logic -

Kindly write your answer here
I loop through the upper and the lower half of the R and use simple maths to determine spacing.

Screenshots(Attach 2 screenshots of your own input and output) -

Attach here

image

  • no input was specified in the requirements.

Checklist:

Eg - If your code follow the below guidelines. Kindly change [] to [x]

All the conditions should be fulfilled for considering your code for merging -

  • I have mentioned the question as comment in my solution file.

  • My code follows the guidelines of this project.

  • I have performed a self-review of my own code.

  • I have commented my code.

  • My code gives the correct output.

  • I confirm that I have not copied the code from anywhere. In case its found that I have copied even after successful merge then I can be banned from the repository and hacktoberfest.

  • I affirm that I strictly follow contributing guidelines and code of conduct.

@SarthakKeshari
Copy link
Owner

@WilsoP,
Kindly mention issue id #313 instead of 313

@SarthakKeshari
Copy link
Owner

@WilsoP,
This PR will be marked as invalid and be closed by 15/10/2021(6PM). Kindly respond by either closing it or making the required changes.

@SarthakKeshari SarthakKeshari added the no response No response to requested changes label Oct 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no response No response to requested changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants