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.

Sign up for GitHub

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

Jump to bottom

codeklavier.py: use default piece #49

Closed
borrob opened this issue Mar 1, 2018 · 1 comment
Closed

codeklavier.py: use default piece #49

borrob opened this issue Mar 1, 2018 · 1 comment
Labels
pending

Comments

@borrob
Copy link
Contributor

borrob commented Mar 1, 2018

running ./codeklavier.py -c default.ini just exits. Perhaps we should print a message that the user pick a piece to play, or run interactively.
@narcode
Copy link
Owner

narcode commented Mar 2, 2018

i was more thinking that there is no default.ini 'by default'. So if it is the first run and default.ini is not yet there then the scripts prompts the user for the interactive createConfig and saves it as default.

I also noticed that just running codeklavier.py without arguments simply exits. It should better print the usage options or showhelp() before exiting.

@narcode narcode added the pending label Mar 2, 2018
borrob added a commit to borrob/codeklavier-python that referenced this issue Mar 2, 2018
@borrob
Add some fluff to the masterscripts (fix issue narcode#49)
3c6da20
@borrob borrob mentioned this issue Mar 2, 2018
Merged
narcode pushed a commit that referenced this issue Mar 2, 2018
@borrob @narcode
Bugfixes (#50)
d74b614 
* Fix configWriter (fixes #47)

* Fix call to configwriter (fixes issue #48)

* Add some fluff to the masterscripts (fix issue #49)

* Default setup is now clavinova

* New doc build
@narcode narcode closed this as completed Mar 2, 2018
narcode added a commit that referenced this issue Apr 30, 2018
@narcode
0.2.1 (#55)
3c48a06 
* first debug session

* Bugfixes (#50)

* Fix configWriter (fixes #47)

* Fix call to configwriter (fixes issue #48)

* Add some fluff to the masterscripts (fix issue #49)

* Default setup is now clavinova

* New doc build

* mystery changes
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
pending
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@borrob @narcode