Skip to content

Conversation

atztogo
Copy link
Collaborator

@atztogo atztogo commented Nov 13, 2022

I hope adding this setup.cfg is harmless for normal cases.

I add setup.cfg to cure my VSCode that can not resolve import spglib from other code by Go to Definition when using pip editable install. This may be due to my specific case: I setup my python environment by conda (conda-forge), but for developing codes in python and installing them by pip install -e .. Those codes work but the codes installed in this way could not be recognized by VSCode. This didn't happen before. I finally found this was fixed (maybe just by luck) by putting setup.cfg. Probably this is related to modern pip (https://pip.pypa.io/en/stable/reference/build-system/pyproject-toml/). I don't understand this theoretically but do empirically.

@codecov-commenter
Copy link

Codecov Report

Base: 85.12% // Head: 85.12% // No change to project coverage 👍

Coverage data is based on head (fc46b10) compared to base (876fb9d).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #203   +/-   ##
========================================
  Coverage    85.12%   85.12%           
========================================
  Files           16       16           
  Lines         1318     1318           
========================================
  Hits          1122     1122           
  Misses         196      196           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@atztogo atztogo merged commit eb2738f into develop Nov 13, 2022
@atztogo atztogo deleted the add-setup-cfg branch November 13, 2022 11:51
@LecrisUT LecrisUT added the enhancement Significant ehancements label Mar 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Significant ehancements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants