-
-
Notifications
You must be signed in to change notification settings - Fork 108
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 a new version #473
Comments
I created #474 for the |
It would be great because CI fails for me when I trying to install Taplo. |
You should always use |
I would like to always install the latest version. |
The problem here is that you can't install with @tamasfe Do you think you would have time for a release? On my side, I'm not sure I do. @panekj what about you? |
I hope the full language server version can be released via NPM too rather than the rather useless version that is currently there. Cheers :) |
I'll take a look at the process tonight but nothing promised. |
I've got time now (I'm currently on holidays until end of next week) so I can take care of release and few other things. |
Cool, thanks!
I don't think so. So yes, trying to follow through #428 would be a start, but I think some things might need input from @tamasfe . |
Are you making any headway with this? There’s now multiple bugfixes to CLI and extension that I’m waiting for. |
Would be nice to have #496 merged before the release, such that we also produce builds for ARM. |
Eventually, you might want to switch to sth. like https://release-plz.ieni.dev/ which automates a lot of things. |
The npm package is also very outdated #480 |
If anyone's interested in forking this and maintaining it with me let me know. |
I've forked it at https://github.com/unrs/taplo but isn't sure to publish it. |
Feel free to also volunteer at maintaining this crate. That's probably the most natural way to start contributing. You would be able to fork later if something is limiting you. If you nevertheless want to fork, which I would understand if you are planning to invest significant effort refactoring, I would suggest contacting @tamasfe to discuss the best way to proceed:
Note that I will most probably step down from maintaining this crate in the coming months (baby is coming so I'll have much less time). Also there's too much Javascript and VS Code related issues in this project which is far from my expertise and my taste. |
@ia0 See #403 (comment) |
As I mentioned I'm strongly against any kind of permission transferring, mainly due to security concerns, and a much better alternative would be to grant more contributors access here or you can fork the project. As for my availability I'm slowly finishing a project at work that took waaaay too much of my time and mental capacity over past year and will have very long holidays time which should let me re-focus on FOSS projects. (I also stepped down from maintenance commitments in other major project) |
The decision of permission transferring is up to @tamasfe anyway, similarly to be added as a contributor. So I guess the next step is to wait to get @tamasfe 's opinion on the matter. An absence of response (say after 1 or 2 months) would mean that forking the project is the best course of action. And in that case I think it should have a new name (because of crates.io but also to avoid transparent trust transfer as @panekj mentioned). |
I'm also a maintainer of many influential projects like https://github.com/prettier/prettier-eslint, https://github.com/prettier/eslint-plugin-prettier, https://github.com/prettier/pretty-quick, etc. I understand how difficult to be trusted as a new maintainer and how difficult to be qualified maintainer, and also forking is always not the best solution which will hurt the community brand. But I still hope I can offer some help here. What is the main blocking now? Can I help rasing a PR for it? Again, I'm very new to Rust community, but I still think we can work together. cc @tamasfe |
I'd say going over all PRs since last release to update the changelogs and the version numbers, such that @tamasfe can just release it. Ideally also test VS Code because that was the problem last time. |
@ia0 If you have permission to invite me as collerbotator, I'd like to help installing some automatic tools which I'm very familiar to use and easy to maintain. So you don't have the permission to release? |
No. And I don't think I can add you either. I'm also going offline for 2 weeks now. The tooling to use would probably be something to discuss. |
OK, I think I'll have to maintain the fork instead then. Do you want to join @unrs org or https://github.com/unrs/taplo repo? |
@JounQin in my opinion you should rename it to something else otherwise people will not find your fork easily as the current one will always show first. |
@o-az We don't need to fork now since after #403 (comment), I'm proud to be a collaborator of this project! ❤️ |
Following up on this as well - any chance of a new VS Code extension release? |
If I'll fix the issues I'm having then maybe |
#449 and #462 are fixing important issues and need to be released. There are other minor changes since the last release.
Before releasing, we should also bump
json_value_merge
to 2.0.0 (and update the code accordingly) because 1.2.0 was a breaking change.The text was updated successfully, but these errors were encountered: