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

Add REUSE.software compliant SPDX tags and pre-commit. #28

Merged
merged 3 commits into from
Jul 21, 2020

Conversation

Flameeyes
Copy link
Contributor

No description provided.

This makes the project comply with the REUSE.software specifications.
The configuration is taken directly from adafruit/Adafruit_CircuitPython_TestRepo.
Comment on lines -41 to +45
- name: Pip install pylint, black, & Sphinx
- name: Pip install pylint, Sphinx, pre-commit
Copy link
Collaborator

Choose a reason for hiding this comment

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

Has there already been a discussion about switching from black to pre-commit? I'm not opposed to it, and after looking into it, it seems like a good option, but I think we may want to discuss it before changing it.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Also, from what I can tell, pre-commit is setup to actually modify the files. We had decided that we shouldn't do that since black and pylint do disagree every so often.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was discussed briefly as part of adafruit/circuitpython#2887

This configuration is taken directly from adafruit/Adafruit_CircuitPython_TestRepo@18e383f

And is already applied to ESP32SPI as well (adafruit/Adafruit_CircuitPython_ESP32SPI@2077d83).

Copy link
Collaborator

Choose a reason for hiding this comment

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

Great. Thanks for linking that

@Flameeyes
Copy link
Contributor Author

Anything left blocking this one?

@tannewt tannewt requested a review from evaherrada July 20, 2020 22:57
@tannewt
Copy link
Member

tannewt commented Jul 20, 2020

@dherrada This is waiting for you.

@evaherrada evaherrada merged commit fc934c1 into adafruit:master Jul 21, 2020
adafruit-adabot added a commit to adafruit/Adafruit_CircuitPython_Bundle that referenced this pull request Jul 24, 2020
@Flameeyes Flameeyes deleted the master branch June 21, 2021 23:01
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