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

Feature #38 add config file #50

Open
wants to merge 20 commits into
base: master
Choose a base branch
from

Conversation

ojob
Copy link

@ojob ojob commented Nov 9, 2017

This is the first commit to implement #38: addition of an options.yml file, that makes parameters persistent on user filesystem.

User can modify the file then restart PhotoCollage; or user can modify the parameters through the settings dialog, then at exit, PhotoCollage will update the settings file.

This commit has been tested with python 2.7 and 3.5, and also with flake8 . and unittest, with no warning/error reported.

Joël Bourgault and others added 16 commits October 26, 2017 13:42
…m filesystem

WARNING: this commit has not been tested (missing GTK dependencies)

Format of the configuration is dict-based. Current implementation will
use the YAML format.

Signed-off-by: Joël BOURGAULT <jbourgault@sii.fr>
Also added usage of built-in `logging` module.

WARNING: this commit has not been tested (missing GTK dependencies)

Signed-off-by: Joël BOURGAULT <joel.bourgault@gmail.com>
WARNING: this has not been tested (missing GTK dependencies)

Signed-off-by: Joël BOURGAULT <jbourgault@sii.fr>
I choose to make interface with other Gtk classes 'simpler', i.e. at a
lower level: these do not access Options instance directly, but directly
to parameters or to `parent` instance.

WARNING: not tested yet (missing GTK dependencies)

Signed-off-by: Joël BOURGAULT <jbourgault@sii.fr>
…ming.

WARNING: not tested yet (missing GTK dependencies)

Signed-off-by: Joël BOURGAULT <jbourgault@sii.fr>
Signed-off-by: Joël BOURGAULT <jbourgault@sii.fr>
WARNING: not tested (missing GTK dependencies)

Signed-off-by: Joël BOURGAULT <jbourgault@sii.fr>
Signed-off-by: Joël BOURGAULT <jbourgault@sii.fr>
Signed-off-by: Joël Bourgault <joel.bourgault@gmail.com>
Signed-off-by: Joël Bourgault <joel.bourgault@gmail.com>
Signed-off-by: Joël Bourgault <joel.bourgault@gmail.com>
And some other minor improvements.

This commit has been tested, both with python2.7.13 and python3.5.3.

Signed-off-by: Joël Bourgault <joel.bourgault@gmail.com>
Signed-off-by: Joël Bourgault <joel.bourgault@gmail.com>
Signed-off-by: Joël Bourgault <joel.bourgault@gmail.com>
Signed-off-by: Joël Bourgault <joel.bourgault@gmail.com>
@ojob
Copy link
Author

ojob commented Nov 10, 2017

Seems that I can't get the newlines right, according to flake8 rules I201 and I202, whatever I try to do following my reading of the documentation.

If anyone gets how I can make this pass the checks, I'm all ears!

Joël Bourgault and others added 4 commits November 10, 2017 10:11
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.

None yet

1 participant