Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP

Loading…

Adding attributes for a lightweight export #3386

Closed
wants to merge 1 commit into from

4 participants

@Ocramius
Collaborator

Simply following @carlosbuenosvinos' idea also for ZF2.

This should basically reduce the size of the zipball produced by github on tagged releases.

Edit: this actually removes ZendTest from the distribution. If the test suite is meant to be shipped with composer, then please feel free to close this PR.

@Maks3w
Collaborator

Sometimes can be good run the test suite of the vendor to try to find if a issue is yours, theirs or some missing PHP extension/configuration

@Ocramius
Collaborator

@Maks3w agreed, but in such cases you just grab zf (clone) and run the tests on the broken system, no?

I'm just wondering if the test suite has to be exported since it's in the composer.json autoload section

@weierophinney

@Maks3w We don't ship the tests currently via composer, pyrus, or the generic distribution packages.

@weierophinney weierophinney commented on the diff
.gitattributes
@@ -0,0 +1,8 @@
+/tests export-ignore
+/vendor export-ignore
+/demos export-ignore
+.gitattributes export-ignore
+.gitignore export-ignore
+.travis.yml export-ignore
+INSTALL.md export-ignore
@weierophinney Owner

Don't we want INSTALL.md?

@Ocramius Collaborator

@weierophinney it's already installed once it's downloaded, no?

@weierophinney Owner

To answer my question, we do -- it details more than obtaining ZF2.

@Ocramius Collaborator

Gotcha. Will remove it from the PR

@weierophinney Owner
@Ocramius Collaborator

Hmm... Didn't you already merge this one and manually remove the INSTALL.md from the .gitattributes?
https://github.com/zendframework/zf2/blob/master/.gitattributes

@weierophinney Owner
@Ocramius Collaborator
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
@carlosbuenosvinos

In order to save space in composer, I have developed https://github.com/carlosbuenosvinos/compify, hope it's usefull

@Ocramius Ocramius referenced this pull request in libracms/zendframework-minimal
Closed

Package is bringing problems to the ZF2 test suite #1

@ghost Unknown referenced this pull request from a commit
@weierophinney weierophinney Merge branch 'feature/3386'
Close #3386
9934049
@ghost Unknown referenced this pull request from a commit
@weierophinney weierophinney Merge branch 'feature/3386' into develop
Forward port #3386
68e1bd8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Jan 9, 2013
  1. @Ocramius
This page is out of date. Refresh to see the latest.
Showing with 8 additions and 0 deletions.
  1. +8 −0 .gitattributes
View
8 .gitattributes
@@ -0,0 +1,8 @@
+/tests export-ignore
+/vendor export-ignore
+/demos export-ignore
+.gitattributes export-ignore
+.gitignore export-ignore
+.travis.yml export-ignore
+INSTALL.md export-ignore
@weierophinney Owner

Don't we want INSTALL.md?

@Ocramius Collaborator

@weierophinney it's already installed once it's downloaded, no?

@weierophinney Owner

To answer my question, we do -- it details more than obtaining ZF2.

@Ocramius Collaborator

Gotcha. Will remove it from the PR

@weierophinney Owner
@Ocramius Collaborator

Hmm... Didn't you already merge this one and manually remove the INSTALL.md from the .gitattributes?
https://github.com/zendframework/zf2/blob/master/.gitattributes

@weierophinney Owner
@Ocramius Collaborator
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
+.php_cs export-ignore
Something went wrong with that request. Please try again.