Skip to content

Conversation

eyeseast
Copy link
Contributor

@eyeseast eyeseast commented Jul 3, 2022

Closes #1764

Small change that adds self.config_dir = config_dir to Datasette.__init__. This will let plugins also use config_dir, if available.

@codecov
Copy link

codecov bot commented Jul 3, 2022

Codecov Report

Merging #1766 (257fd24) into main (9f1eb0d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main    #1766   +/-   ##
=======================================
  Coverage   91.67%   91.67%           
=======================================
  Files          36       36           
  Lines        4658     4659    +1     
=======================================
+ Hits         4270     4271    +1     
  Misses        388      388           
Impacted Files Coverage Δ
datasette/app.py 94.27% <100.00%> (+<0.01%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9f1eb0d...257fd24. Read the comment docs.

@simonw
Copy link
Owner

simonw commented Jul 18, 2022

This is a good idea, thanks.

@simonw simonw merged commit 0136917 into simonw:main Jul 18, 2022
simonw added a commit that referenced this pull request Aug 14, 2022
simonw added a commit that referenced this pull request Aug 14, 2022
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.

Keep track of config_dir in directory mode (for plugins)
2 participants