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

[ENH] Orange.misc.environ config #4576

Merged
merged 2 commits into from
Apr 3, 2020
Merged

Conversation

ales-erjavec
Copy link
Contributor

Issue

Ref gh-4264

Description of changes
  • Read configured paths from a .cfg file. The cfg file can use and define install prefix relative paths
  • Use the defined paths to override the default paths for storing widget and canvas settings.
Includes
  • Code changes
  • Tests
  • Documentation

@codecov
Copy link

codecov bot commented Mar 30, 2020

Codecov Report

Merging #4576 into master will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #4576      +/-   ##
==========================================
+ Coverage   83.59%   83.60%   +0.01%     
==========================================
  Files         281      276       -5     
  Lines       56131    55406     -725     
==========================================
- Hits        46921    46323     -598     
+ Misses       9210     9083     -127     

@thocevar thocevar self-requested a review March 31, 2020 15:54
Copy link
Contributor

@thocevar thocevar left a comment

Choose a reason for hiding this comment

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

This will enable us to set the location of widget settings for a portable version
(#4264). I've managed to set up a config file and read/write settings to a specified relative path.

@thocevar thocevar merged commit 94c0178 into biolab:master Apr 3, 2020
@ales-erjavec ales-erjavec deleted the orange-config branch August 5, 2020 11:43
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.

3 participants