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

add new c pattern #364

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

karanghuge01
Copy link

@karanghuge01 karanghuge01 commented Oct 17, 2024

fix : #357

Description
This pull request introduces a new C pattern: Hollow Square Star Pattern. The pattern demonstrates how to print a hollow square shape made of asterisks (*) based on the specified number of rows.

Time Complexity: O(n^2) - There are nested loops for rows and columns.
Space Complexity: O(1) - The program uses a constant amount of space.

Checklist

  • I have tested the code and ensured it works as expected.
  • I have updated the documentation (if applicable).
  • I have followed the project's coding style.

Copy link
Owner

@panditakshay402 panditakshay402 left a comment

Choose a reason for hiding this comment

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

@karanghuge01 Kindly follow all the rules from Readme.md to get PR accepted

@karanghuge01
Copy link
Author

I have carefully gone through the README.md file and followed the instructions to the best of my understanding. Since I am a beginner, I wanted to ask if there is anything I might have missed or overlooked in my pull request. I would appreciate your guidance on what needs to be done so I can ensure my PR is ready to be merged.

@panditakshay402
Copy link
Owner

panditakshay402 commented Oct 18, 2024

I have carefully gone through the README.md file and followed the instructions to the best of my understanding. Since I am a beginner, I wanted to ask if there is anything I might have missed or overlooked in my pull request. I would appreciate your guidance on what needs to be done so I can ensure my PR is ready to be merged.

@karanghuge01 Yes Sure!!

there is some checklist (WorkFlow) created in this repo to check Rules:
1.Follow the user and star the repo.
2. Contributions must be meaningfulls. (dont just add sorting,searching, etc code).
3. commit must be greater that 50 lines.

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.

Add more patterns c programs
2 participants