-
Notifications
You must be signed in to change notification settings - Fork 25
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
Known issues? Looking for more maintainers? #13
Comments
And to be clear: if you're primarily looking to make forward progress with this fork, and were looking for more casual collaborators I would put my hat in the ring. -- If you'd rather not have a lot of primary development in this fork, then that's fine too -- I wouldn't want to push you beyond your vision here. I'm currently a regular OSS collaborator on github.com/danger/danger-js and few other projects. |
Hi @fbartho thanks for dropping by! Originally my plan was to only include @blutorange's work to avoid sorting side-effect imports, because I saw that as a critical bugfix that the maintainers of the trivago project disagreed with. Then @blutorange and I made some other fixes, and I found some nice PRs from the upstream project that seemed stalled out that I brought over. And, well, instead of needing to worry about staying up-to-date with upstream, we've ended up diverging. I think this is fine. I'd like to continue being able to pull in changes (either proposed or merged) from upstream when possible, but merging this back into the Trivago is a non-goal.
I don't know of any issues that remain in this project that are dangerous, and I believe it to be safe to apply to a large project. I use it in production at work, without problems.
I can't think of anything, but maybe @blutorange has something in mind?
Sure! I'd be happy for the help. Immediate tasks that come to mind would be reviewing #12, fixing #9 (minor annoyance), or going through the issues / PRs in the trivago repo to see if there are any that we could consider fixing here. Or, if you have other thoughts on improvements that could be made, feel free to open an issue and we can discuss. Cheers! |
Oh, and if you'd like to submit a PR to update the wording in the README, I'd be open to that as well. It's not exactly accurate anymore as you pointed out. |
Since this is "just" a linter, I'd consider any possible risk as rather low. Said that, this trivago/prettier-plugin-sort-imports#116 is something we should check whether that issue still exists and fix it, non-deterministic ordering may result in different formatting depending on the build environment.
Yes, as IanVS said, any help is welcome.
Personally I wouldn't mind if they wanted to merge this back, but that doesn't seem too likely at this point. |
At the top of the readme you mention this is a best-effort fork of the
trivago/prettier-plugin-sort-imports
but that it may not stay up-to-date forever.I've been hesitant to use
trivago/...
because of all of the breaking issues, but it seems like on each issue I've examined that I would consider a dealbreaker I find a comment from @IanVS saying that this fork fixes that problem :DSo I guess my questions are:
trivago/...
that still need to be fixed in this fork, or that you would consider danger for somebody trying to apply this plugin to a large codebase?The text was updated successfully, but these errors were encountered: