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

We forked this package and have fixed many bugs #47

Open
alexeyinkin opened this issue May 9, 2023 · 11 comments
Open

We forked this package and have fixed many bugs #47

alexeyinkin opened this issue May 9, 2023 · 11 comments

Comments

@alexeyinkin
Copy link

This project seems abandoned, so we forked it and did the following:

The new package is published as:

  1. https://pub.dev/packages/highlighting
  2. https://pub.dev/packages/flutter_highlighting

We intend to maintain our fork.
See the migration guide: https://github.com/akvelon/dart-highlighting/blob/main/doc/migrating.md

(Posting this here as there seems to be no other way to inform the users of the package who are in need of changes.)

@pd4d10
Copy link
Member

pd4d10 commented May 9, 2023

Hi @alexeyinkin, just see this and your previous issue #38. Thanks for the work to improve it!

This project seems abandoned

As you said, I really don't have much time to maintain this package at the moment, so I also think it's a more appropriate choice to hand it over to a team that has enough time to maintain it.

Considering the existing users, perhaps I can add you and your coworkers as admin (for both GitHub repo and pub.dev package), and you can directly use this package name. Then existing users do not need to do additional migration work, they just need to upgrade the version.

What do you think?

@alexeyinkin
Copy link
Author

@pd4d10 this would be awesome!

For pub.dev, please add [email protected] as admin of the two packages on pub.dev. I can see that they are published as 'Unverified publisher'. This means that it may be easier to do the reverse: I can add your pub.dev email as admin of my publisher, and you can transfer the packages there. If it is easier for you, please drop your email to me at my email above.

Regarding GitHub, we would rather maintain our own fork because we have a history there. What we can do in this repository is write in the README.md an encouragement to migrate and all the new links. For that, please make me an admin (or whatever it is called on GitHub).

@pd4d10
Copy link
Member

pd4d10 commented May 9, 2023

we would rather maintain our own fork because we have a history there

Not quite sure about what you mean, I guess a simple git push should sync the git history, and you and your coworkers' contributions could also be recognized.

If you're concerned about the organization name, maybe we can move this repo to a new organization, such as something like "highlight-dart", just like @highlightjs does.

What do you think?

@alexeyinkin
Copy link
Author

OK. My employer would want it under their https://github.com/akvelon organization. Please transfer it there.

@pd4d10
Copy link
Member

pd4d10 commented May 9, 2023

My employer would want it under their https://github.com/akvelon organization

OK. Personally, I prefer it to be maintained by the community, but I also fully understand your request, if these work were done during the time working for the company.

Please give me some time to consider it.

Anyway, thanks very much for your work to improve it!

@alexeyinkin
Copy link
Author

@pd4d10 can you add me on LinkedIn? https://www.linkedin.com/in/alexey-inkin-784b7371/
Communication seems to take long otherwise.
This is just to keep in touch because I no longer work for Akvelon.

@amake
Copy link

amake commented Sep 24, 2023

@alexeyinkin Now that you are no longer working for Akvelon, can you update us about the status of flutter_highlighting? Is there a commitment to maintain it (by who)?

@alexeyinkin
Copy link
Author

In Akvelon, @nausharipov is now in charge of Flutter. If anyone knows the roadmap, it is him.

@busslina
Copy link

busslina commented Dec 1, 2023

Would you adopt this PR?
#50

@nausharipov
Copy link

@busslina, yes. Thank you. I've replied here: akvelon/dart-highlighting#74 (comment)

@amake
Copy link

amake commented Oct 29, 2024

A year later, the fork appears to be abandoned 🫤

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

5 participants