Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

move test and avoid packaging them#1028

Merged
Byron merged 4 commits intogitpython-developers:masterfrom
priv-kweihmann:move-tests
Jul 12, 2020
Merged

move test and avoid packaging them#1028
Byron merged 4 commits intogitpython-developers:masterfrom
priv-kweihmann:move-tests

Conversation

@priv-kweihmann
Copy link
Contributor

Move test files out of git module and into the root dir.
This enables it to ignore them while setuptools packaging and is resulting in a way smaller release package.

Closes#1027

airone01 reacted with heart emoji
This should ensure that tests are NOT packaged intorelease package by setuptools, as tests are developmentonly+ fixtures after movingSigned-off-by: Konrad Weihmann <kweihmann@outlook.com>
by using exclude feature of find_packages.Plus remove now obselete package_data settingSigned-off-by: Konrad Weihmann <kweihmann@outlook.com>
and remove all previously used test related settingsSigned-off-by: Konrad Weihmann <kweihmann@outlook.com>
Signed-off-by: Konrad Weihmann <kweihmann@outlook.com>
@priv-kweihmannpriv-kweihmann changed the titlemove test and avoid packaging thmemove test and avoid packaging themJul 10, 2020
@ByronByron self-requested a reviewJuly 12, 2020 04:07
@ByronByron merged commit0238e6c intogitpython-developers:masterJul 12, 2020
@Byron
Copy link
Member

I am absolutely amazed and am super thankful for this PR and the effort that undoubtedly went into it.
It will have a massive impact to everyone, given that GitPython is downloaded a lot!

Will cut a new release right away!

priv-kweihmann and airone01 reacted with thumbs up emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@ByronByronAwaiting requested review from Byron

Assignees

No one assigned

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Don't install unittest code by default

2 participants

@priv-kweihmann@Byron

[8]ページ先頭

©2009-2026 Movatter.jp