-
Notifications
You must be signed in to change notification settings - Fork 91
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
Release with latest Angular support #121
Comments
Sorry, no ETA, Haithem is the only one who can release packages this moment. |
He seemed to stop being active a while after a year ago. I hate to say this, but should we maybe consider forking and continuing the package ourselves? I've sent him an email around a month ago asking about the dependency updates before I decided to do them myself, but he didn't answer. And I think that, with the due reference, we should consider continuing the project ourselves @odahcam |
Yeah, this is a recorrent question now. I won't be an active contributor as well, but I can in fact help maintain. @PowerKiKi is a good contributor and his opinion matters in that question. |
@Heatmanofurioso, we've been on the verge of forking a few months ago, but have been saved in extremis. But since then, the whole thing died down again. So I recently made up my mind to migrating away from this lib. My opinion on things didn't change since then. |
I don't see myself having much time to be a main contributor, but I could easily also take part in helping maintain the library. I think the library has some decent potential to be kept alive, and the biggest issue in forking it would probably be bringing attention to the forked version. "That, and I'd like to avoid it as much as possible, since I am a strong supporter of not trying to take ownership of something I shouldn't. But HaithemMosbahi seems to have gone inactive, and I think it's what we should do". @odahcam Would you like to take ownership of it then? Or should I if no one else volunteers? EDIT: I've tried contacting HaithemMosbahi one last time also, to see if he might share ownership of the repo, but if he doesn't, I feel that the opensource spirit would be for us to either drop it, or work on it |
I would take ownership if needed, no problem. But I would just maintain the repsitory and probably wouldn't implement many things. |
Sounds great @odahcam! |
If you can get a hold of HaithemMosbahi, I strongly suggest you ask for total control on this repository and the related npm registry (for one or both of you guys), or else fork the whole project. Anything else would be a waste of time for both parties. |
Yeah, we need someone else to keep updating this repo & publish new versions. @HaithemMosbahi can you please give access to other people to improve this library? That would be great! Thanks! |
fingers crossed... |
Well. Apparently the owner hasn't had time to reach out to us and help us out with this after that initial talk. As I said.. I'm not much in favour of "taking someone's work", but in the open-source spirit, I would like to maintain this project, and also to keep on using it |
Fork right now, with the appropriate attribution, and be done with it. The MIT license allows you do pretty much anything you want. And it's been months that several people tried to establish a working collaboration, but it never happened, because the owner did not share enough accesses. Fork, rename, make a respectful announcement around here, and move the community to something that is maintained (collaboratively). |
We can create a package ngx-avatars (plural) and that's it. |
Anyone opposes on me making a fork on that namespace and attempting to make the package on npm? |
Also... any suggestions for the name? 😆 |
Just put ngx-avatars and let's give access to at least two contributors to the package (I'm saying, just to speed up things). If you need help please let me know. |
The repo has been created at https://github.com/Heatmanofurioso/ngx-avatars EDIT: |
In case anyone is interested, the package is published and working at ngx-avatars |
Maybe we could put a note in the README in this repo, telling that is deprecated and the project will continue as ngx-avatars. |
@odahcam If you agree, could you see to it? |
Hello guys, Sorry for not being active for a long period. I just published a new version with the changes that you merged over the last months. Who would be interested in being added to the npm package maintainers list, that way I don't block the release of new versions when I am not available. |
Hi Haithem. I'll gladly deprecate that child project if we can keep this one up :) |
Great! Thank you for your help |
@PowerKiKi Can you please also share your npm user? It could be great to have at least 2 users in addition to @HaithemMosbahi |
I am https://www.npmjs.com/~powerkiki. You should definitely add https://www.npmjs.com/~odahcam who has been helping here even longer than me, and https://www.npmjs.com/~heatmanofurioso who owns the new fork. |
Sounds good! I will add you as maintainers. |
@PowerKiKi @odahcam @Heatmanofurioso I just added you as maintainers to the npm package. You should have received an invite by now. |
Same here. Although I'm not sure if you show up as a collaborator only once you effectively collaborate "do a publish". Thank you for your time and attention @HaithemMosbahi I'm gonna do a PR taking Angular 11.2.x in consideration here "once I have time", and once we can support the latest version, I'm going to deprecate my forked project, and point it back here |
@HaithemMosbahi is there a way you can give us permissions to change the secrets of this repository? That would allow us to configure GitHub Actions to test and auto-release the master branch! Edit: I forgot to thank you @HaithemMosbahi for giving us that privilege, thanks! |
I recommend creating a GitHub org, transfer ownership of this repository to that org, and then increase access privilege (to admin) for some of us. |
This convo seems to be addressed on #125 |
Hi @HaithemMosbahi. Any possible ETA on the release of the latest changes?
If there is any issue, or anything I can help with towards it, I'd gladly take up any issue
The text was updated successfully, but these errors were encountered: