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 arguments to cmd_new.py to allow no input creation of commands #92

Merged

Conversation

crushingCodes
Copy link
Contributor

To test use the new functionality use:
keep new --cmd hello1 --desc "echo 1" --alias " "

or the original command to ensure it still works like original
keep new

requirements.txt Outdated
@@ -1,4 +1,4 @@
click
requests
terminaltables
PyGithub==1.43.7
PyGithub
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I needed to rm the pinned version number here to avoid the following error:

error in PyGithub setup command: use_2to3 is invalid

Copy link
Collaborator

Choose a reason for hiding this comment

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

is it possible something is not working with your local python setup?

Copy link
Collaborator

@rahuliyer95 rahuliyer95 left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, everything looks good but the PyGithub requirement change

requirements.txt Outdated
@@ -1,4 +1,4 @@
click
requests
terminaltables
PyGithub==1.43.7
PyGithub
Copy link
Collaborator

Choose a reason for hiding this comment

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

is it possible something is not working with your local python setup?

@crushingCodes crushingCodes force-pushed the add-arguments-to-create branch from f917430 to 1ee0952 Compare May 20, 2023 01:44
@crushingCodes
Copy link
Contributor Author

Thanks for reviewing @rahuliyer95, i reverted the requirements.txt.

@crushingCodes crushingCodes requested a review from rahuliyer95 May 20, 2023 01:47
@rahuliyer95 rahuliyer95 merged commit e6eadb9 into OrkoHunter:master May 20, 2023
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.

2 participants