Skip to content

Conversation

@devRawnie
Copy link

Description

Started the code for making the entire repository menu driven

Fixes #831

Have you read the Contributing Guidelines on Pull Requests?

  • Yes
  • No

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • This change requires a documentation update

Checklist:

  • My code follows the style guidelines(Clean Code) of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have created a helpful and easy to understand README.md
  • My documentation follows Template for README.md
  • My changes generate no new warnings
  • I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Congratulations!! for making your first PR at Amazing-Python-Scripts, our mentors will review it soon.

@kaustubhgupta kaustubhgupta added the draft-pr This is draft-pr and not the final PR to be merged label May 8, 2021
@devRawnie devRawnie marked this pull request as ready for review May 8, 2021 15:44
@kaustubhgupta kaustubhgupta added hold Needs a second thought and removed draft-pr This is draft-pr and not the final PR to be merged labels May 11, 2021
@kaustubhgupta
Copy link
Contributor

@devRawnie I will review your PR after the database updation script works fine. Also, I am seeing a lot of issues will be incoming in this demo script. The requirements are the major issue plus the arguments. We will figure out all the issues

@devRawnie
Copy link
Author

@devRawnie I will review your PR after the database updation script works fine. Also, I am seeing a lot of issues will be incoming in this demo script. The requirements are the major issue plus the arguments. We will figure out all the issues

Sure

@kaustubhgupta
Copy link
Contributor

@devRawnie Now the action is working properly and we can work on your issue. I would like you to pull the latest changes in your local system and then report what challenges you are facing. If you want, you can create separate issue other than the master issue (#831) for better clarity

@devRawnie
Copy link
Author

Ok I am on it. Will revert back shortly

@devRawnie
Copy link
Author

Kindly refer to #1071 issue for problems related to the menu driven script.

@github-actions
Copy link

Due to inactivity this pull request has been marked as stale.

@github-actions github-actions bot added the Stale PRs with no updates label May 26, 2021
@kaustubhgupta
Copy link
Contributor

Reason for closing: #1071 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

hold Needs a second thought Stale PRs with no updates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Making the Whole Project/Repo Menu-Driven

2 participants