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

use pkgutil to load resources to resolve pyinstaller issues #81

Merged
merged 5 commits into from
Feb 23, 2021

Conversation

barrust
Copy link
Owner

@barrust barrust commented Jan 7, 2021

Another method to fix #64 utilizing pkgutil

@codecov-io
Copy link

codecov-io commented Feb 22, 2021

Codecov Report

Merging #81 (fb61d6c) into master (e666ad9) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
+ Coverage   99.19%   99.21%   +0.01%     
==========================================
  Files           4        4              
  Lines         249      255       +6     
==========================================
+ Hits          247      253       +6     
  Misses          2        2              
Impacted Files Coverage Δ
spellchecker/spellchecker.py 99.08% <100.00%> (+0.02%) ⬆️
spellchecker/utils.py 100.00% <100.00%> (ø)

@barrust barrust merged commit 5e03e50 into master Feb 23, 2021
@barrust barrust deleted the fix-64-option2 branch February 23, 2021 02:37
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.

ValueError: The provided dictionary language (en) does not exist!
2 participants