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 .gitattributes file to ignore paths when exporting #49

Closed
wants to merge 1 commit into from

Conversation

michaeldyrynda
Copy link

Create a .gitattributes file, and add paths with the export-ignore flag set, so they're not pulled down as part of a project's dependencies via Composer.

@VincentLanglet
Copy link

Hi @TingDaoK, would it be possible to review this PR or to provide another one with gitattributes file ?

It would be great to not install things like the tests folder when require for this library.

Thanks

@TingDaoK
Copy link
Contributor

Hi @TingDaoK, would it be possible to review this PR or to provide another one with gitattributes file ?

It would be great to not install things like the tests folder when require for this library.

Thanks

I already did, it's #89 and released with the new change.

@TingDaoK TingDaoK closed this Mar 23, 2023
@VincentLanglet
Copy link

Thanks, and sorry, seems like I was only at the 1.0.4 version. I'll update.

@TingDaoK
Copy link
Contributor

no worries.

Seems like the new github tags were not picked by packagist. https://packagist.org/packages/aws/aws-crt-php
You will need to use the dev-main.

Should it be automatically pulled by packagist? https://laravelpackage.com/15-publishing.html#releasing-a-new-version

@TingDaoK
Copy link
Contributor

Oh, okay. I guess it has not updated since 3/3

This package is auto-updated.

Last update: 2023-03-03 17:24:42 UTC

@VincentLanglet
Copy link

This is weird. I dunno why the package is not updated on packagist...

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.

None yet

3 participants