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

Ignore files when exporting package #3204

Closed
wants to merge 1 commit into from
Closed

Ignore files when exporting package #3204

wants to merge 1 commit into from

Conversation

simonschaufi
Copy link

This commit is part of a campaign to reduce the amount of data transferred to save global bandwidth and reduce the amount of CO2. See Codeception/Codeception#5527 for more info.

@simonschaufi
Copy link
Author

What about CodeSniffer.conf.dist? Can this file also get excluded?

This commit is part of a campaign to reduce the amount of data transferred to save global bandwidth and reduce the amount of CO2. See Codeception/Codeception#5527 for more info.
@jrfnl
Copy link
Contributor

jrfnl commented Jan 30, 2021

Related to #548, #1908, #2991, #3176 - the change for tests will not be accepted in the 3.x branch. The tests will be export-ignored in the 4.x branch.

The update for .github/ is justified. This is a recently new directory.

@simonschaufi
Copy link
Author

Ok I see but I really wonder why people depend on third party tests... When can I expect a 4.0 release?

gsherwood added a commit that referenced this pull request Jan 31, 2021
gsherwood added a commit that referenced this pull request Jan 31, 2021
@gsherwood
Copy link
Member

I've added the .github dir to the export ignore list. Thanks for pointing that out.

The test dirs are ignored in the 4.0 branch but I do not have a release date for that version.

@gsherwood gsherwood closed this Jan 31, 2021
@simonschaufi simonschaufi deleted the gitattributes branch January 31, 2021 01:34
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