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 composer.json #76

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Add composer.json #76

wants to merge 2 commits into from

Conversation

j0k3r
Copy link
Collaborator

@j0k3r j0k3r commented Aug 7, 2015

It'll ease integration of all these config files inside other PHP project

@fivefilters
Copy link
Owner

Thanks Jeremy, looks really useful to have in the repository. I want accept this, but I can't until Full-Text RSS is updated to ignore the presence of this file in its auto-update routine. At the moment it ignores files based on a list of extensions, but .json isn't one of them. Until I add that and test it, I can't accept this - otherwise it'll break auto-updates for users who've set it up.

@j0k3r
Copy link
Collaborator Author

j0k3r commented Aug 7, 2015

@fivefilters no problem. When do you plan to release the next update of Full-Text RSS ?

@fivefilters
Copy link
Owner

Jeremy, not sure about next update yet. And thinking about it again, even if the next FTR update handles this properly, there'll still be users of old versions relying on the repository for auto-updates. So probably won't add it until at least one more version is out.

@j0k3r
Copy link
Collaborator Author

j0k3r commented Aug 10, 2015

Ok no problem.
Maybe you should handle file verification in an other way: only grab *.txt files instead of building a list of not allowed extensions.

@jtojnar
Copy link
Contributor

jtojnar commented Jan 14, 2016

Any progress on this? It would make developing selfoss much easier.

@jeroenseegers
Copy link

Nothing on this, still?

@johnbillion
Copy link

This would be really handy for a project of mine. Any chance that this can go in?

@nijel
Copy link
Contributor

nijel commented Oct 22, 2018

I guess that's reason why https://packagist.org/packages/j0k3r/graby-site-config exists...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants