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

No active maintainer #229

Closed
merk opened this issue Jul 7, 2014 · 34 comments
Closed

No active maintainer #229

merk opened this issue Jul 7, 2014 · 34 comments

Comments

@merk
Copy link
Member

merk commented Jul 7, 2014

We are seeking someone who would like to assist in general maintenance of this bundle, triaging tickets and assisting in reviewing and okaying PRs.

@tgalopin
Copy link
Contributor

tgalopin commented Jul 9, 2014

I can probably help. I don't have much time but I use the bundle a lot and I could help fix some issues.

Are you searching for help or for a complete maintainer? Is there a way to talk with the ex maintainers for a better understanding of the bundle internals?

@csvan
Copy link

csvan commented Jul 11, 2014

I would volunteer as well. We use the bundle a lot in internal project. I am not too familiar with the actual internals, but can catch up as necessary.

@ajaypatelbardoli
Copy link

me too.

@tgalopin
Copy link
Contributor

Let's do something then, we are three, we can work on the current issues.

@merk you seem to have worked on a 3.0-dev branch, but I can't see it anywhere. Is it done or did you give up on it?

@ojhaujjwal
Copy link

I can help too if this bundle is splited into component.

@csvan
Copy link

csvan commented Jul 16, 2014

I am sitting as project manager at work at the moment, so time is relatively sparse. Will read up on the internals and existing issues and see what I can start doing.

@merk
Copy link
Member Author

merk commented Jul 16, 2014

I was working on something previously but to be honest I'm not sure where I was at or if it was worth using it to continue.

https://github.com/merk/FOSMessageBundle/tree/refactoring

@merk
Copy link
Member Author

merk commented Jul 16, 2014

Regarding what we're after - any help is better than none.

@merk
Copy link
Member Author

merk commented Jul 16, 2014

If @zocimek is happy to continue and with the help offered here, we can probably remove the notice in the readme.

@zocimek
Copy link
Contributor

zocimek commented Jul 16, 2014

I think we can continue maintain bundle. For now I'm a little bit busy but I think I can find some time to review changes and fix small bugs. With help it should be much better.

@tgalopin
Copy link
Contributor

@zocimek As you know better the bundle than us, you should probably assign us some tasks to do, as, for instance, I don't know which are still valid or not (some issues are older than a year).

@sylvaindeloux
Copy link

Hi,
I think this project is dead :/
Is there any other repo or fork to replace this one?

@tgalopin
Copy link
Contributor

I agree a change is necessary but I think changing maintainer would be better than forking. The problem with forking is that we would lose the "renown" of FOS and the MessageBundle.

@redayoub
Copy link

+1 for changing the maintainer

@tgalopin
Copy link
Contributor

The thing is we need a current maintainer to change. Is there anyone reading? Is the bundle really dead?

@merk
Copy link
Member Author

merk commented Apr 21, 2015

The trouble is the current FOS members dont have time to maintain this bundle (and probably arent using it).

Ultimately, to become promoted to maintainer we'd want to see some activity: forking and merging outstanding PRs or making improvements would indicate this - then that code could be merged back here.

My opinion is that we dont want to keep appointing maintainers who will disappear again.

@AlexGodard
Copy link

Just dropping here to mention I planned on using this bundle. So changes would be appreciated yes! Is the current version a working one ?

@tgalopin
Copy link
Contributor

@merk So should we fork it and work on it on our own then? I'm using this bundle pretty regularly and some ideas (as making a library as a base of the bundle for instance) are great ideas IMO. There are a lot of possibilities :) .

I agree we don't want temoporary maintainers, but for the moment the bundle does not evolve at all, so even a bit of dynamic would be great.

@merk
Copy link
Member Author

merk commented Apr 21, 2015

My suggestion at this point would be to fork the bundle, work on your fork for some time making changes you think would improve the bundle and propose to become the maintainer linking to your work - that means your work can be merged into a new major release of the bundle here.

Alternatively, sending PRs that fix peoples issues or add new functionality and pinging me or @zocimek and in the short term we can merge them.

My personal approach here would be to add some functional testing because there is very little of that in the bundle.

@tgalopin
Copy link
Contributor

I forked the bundle and I'll start to work on it there: https://github.com/tgalopin/FOSMessageBundle

However, on the main repo, some issues could be closed such as:

Perhaps now is the moment to think of a new version of the bundle based on the current one concepts but using a framework independant library. I think it could be a great project but such a changement require motivated people. Are there motivated people here ready to give a hand (for the library, the bundle would be very light)?

@redayoub
Copy link

@tgalopin I'm interested, what kind of help do you need?

@tgalopin
Copy link
Contributor

The kind of help to rewrite the bundle as a library :)

I think the steps would be:

  1. Isolate the concepts of the current bundle and what we want to do in general
  2. Implement the basis of the library using these concepts
  3. Discuss more on what we want to add as feature

I started a repository here to discuss the library: https://github.com/tgalopin/FOSMessage
I think we should have the discussions in the issues of this repo (https://github.com/tgalopin/FOSMessage/issues).

So the main and first step for the moment is to review all the features the current bundle have and what we want to add / remove to these features. I started a discussion here: https://github.com/tgalopin/FOSMessage/issues/1

@tgalopin
Copy link
Contributor

I started to explain what I think the library should be, but I need opinions, thoughts and critics:
https://github.com/tgalopin/FOSMessage/issues

I also need feedbacks of how you currently use the bundle: what you need, what you think that should be implemented, things like that. The new version won't be fully compatible with this version so we can invent new things and resolve new problems.

Thanks by advance!

@tgalopin
Copy link
Contributor

I ported almost all the concepts of the current bundle in the library. I changed some things and add others to match with the idea of the library (it's not a bundle anymore after all :) ):

https://github.com/tgalopin/FOSMessage

I am still currently working a lot on the code, don't hesitate to give a hand. For the moment, the next tasks are:

  1. Create tests for the current code
  2. Implement drivers for Doctrine, Propel, Doctrine ODM
  3. Review the current code for mistakes / typos in order to match PSR-2 and have clear comments ;
  4. Start to write a doc on how to use the library without Symfony

What is great with this library is that it has only one light dependency (on Doctrine collections), it's very easy to reuse it without any framework.

I didn't implement things like SPAM and security: I didn't implement SPAM for the moment in order to stabilize the library before, I won't implement security in the library as it's the role of the framework / developer.

Please give feedbacks, it's my first shared library, I want to improve it as much as possible :) !

Thanks by advance!

@redayoub : do you use Propel or Doctrine ODM?

@KhanMaytok
Copy link

Can I help?

@tgalopin
Copy link
Contributor

tgalopin commented Jun 5, 2015

Yes, don't hesitate! Either by contributing on the library I'm working on or by contributing to this repository (but I can't be sure your pull request will be merged any time soon).

@smilesrg
Copy link

smilesrg commented Jul 9, 2015

@tgalopin it's a good idea to write documentation for your library.

@gperdomor
Copy link

@tgalopin great work... But we need some documentation :D

@tgalopin
Copy link
Contributor

I'm working on it, it should be available soon in an early version :) .

@tgalopin
Copy link
Contributor

I worked a lot on the library yesterday and today, it's improving a lot while far from being ready for the moment.

If you want to try it and play with it, you can use the examples:

https://github.com/tgalopin/FOSMessage/blob/master/Resources/doc/examples/full-example.php
https://github.com/tgalopin/FOSMessage/tree/master/Resources/doc/examples/entities

The concepts of the library are the same as the bundle, but the code is not exactly the same. I'll write a beginning of documentation as soon as possible (probably this night and tomorrow).

I'm very interested by your opinion about the library and your ideas about it, please don't hesitate to comment, propose ideas and contribute! It's a big project and I really need inputs :) .

@tgalopin
Copy link
Contributor

Here is a beginning of documentation:

https://github.com/tgalopin/FOSMessage/tree/master/Resources/doc

Please try the library, I really would like to have inputs!

@tgalopin
Copy link
Contributor

@merk : do you think you could put a link to my library in the README of the bundle as the new version in development? It would improve its visibility and people would be aware of its existence.

@lsmith77
Copy link
Member

lsmith77 commented Aug 3, 2015

@dbu
Copy link

dbu commented Sep 21, 2015

as per FriendsOfSymfony/friendsofsymfony.github.io#54 we have new maintainers. thanks @tgalopin and @ChristianRiesen!

@dbu dbu closed this as completed Sep 21, 2015
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

No branches or pull requests