Skip to content

Ease of use changes#55

Open
MllwAlms wants to merge 2 commits into
medvm:mainfrom
MllwAlms:patch-2
Open

Ease of use changes#55
MllwAlms wants to merge 2 commits into
medvm:mainfrom
MllwAlms:patch-2

Conversation

@MllwAlms

@MllwAlms MllwAlms commented Feb 1, 2022

Copy link
Copy Markdown
  • URL connection success notice
  • Replace input with argv
  • Automates license header update using uncurl and URL

- URL connection success notice
- Replace input with argv
- Automates license header update using uncurl and URL
@pythonkenyard

pythonkenyard commented Feb 2, 2022

Copy link
Copy Markdown

You should allow both options for situation where users dont know to add it as sys arg.

try:
MDP_URL = sys.argv[1]
lic_url = sys.argv[2]
except:
MDP_URL = input('\nInput MPD URL: ')
lic_url = input('License URL: ')

edit: comments dont allow indent formatting it seems

@MllwAlms

MllwAlms commented Feb 4, 2022

Copy link
Copy Markdown
Author

Added

You should allow both options for situation where users dont know to add it as sys arg.

try: MDP_URL = sys.argv[1] lic_url = sys.argv[2] except: MDP_URL = input('\nInput MPD URL: ') lic_url = input('License URL: ')

edit: comments dont allow indent formatting it seems

Added

Added option for user input. Doesn't force argv.
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