Skip to content

Commit

Permalink
Just put everything into the sdist so everyone is happy
Browse files Browse the repository at this point in the history
  • Loading branch information
@hynek
hynek committed Dec 7, 2018
1 parent 95fec95 commit 2b31a30
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions MANIFEST.in
View file
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
include LICENSE *.rst *.toml .readthedocs.yml .pre-commit-config.yaml

# Don't package GitHub-specific files.
exclude .github/*.md .travis.yml codecov.yml
include LICENSE *.rst *.toml *.yml *.yaml
graft .github

# Stubs
include src/attr/py.typed
Expand All @@ -10,7 +8,6 @@ recursive-include src *.pyi
# Tests
include tox.ini .coveragerc conftest.py
recursive-include tests *.py
recursive-include .github *.rst

# Documentation
include docs/Makefile docs/docutils.conf
Expand Down

3 comments on commit 2b31a30

@webknjaz
Copy link
Member

Choose a reason for hiding this comment

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

Why not just include git-tracked files instead of maintaining manifest manually?

@hynek
Copy link
Member Author

@hynek hynek commented on 2b31a30 Dec 15, 2018

Choose a reason for hiding this comment

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

that would require a setup_requires dependency, right?

@webknjaz
Copy link
Member

Choose a reason for hiding this comment

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

Usually yes, but I can see how I could work around it if needed. I can send a PR if you think that it's something you'd want to consider.

Please sign in to comment.