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

v2.0.0 #74

Closed
5 of 6 tasks
rafaelrinaldi opened this issue Jul 10, 2018 · 13 comments
Closed
5 of 6 tasks

v2.0.0 #74

rafaelrinaldi opened this issue Jul 10, 2018 · 13 comments
Assignees
Labels
🚀 enhancement performance, UX or maintainability
Milestone

Comments

@rafaelrinaldi
Copy link
Collaborator

rafaelrinaldi commented Jul 10, 2018

I think we're almost good to go on a v2.0.0 release.

Current checklist:


I'd love to hear your thoughts on what we have right now and would also appreciate if someone can help with the pending items.

Hope you guys are doing well!

@canhduong28
Copy link

Hello,

Thank you all for an awesome plugin.

I have just upgraded to the latest version with Vim mode support. Is there any way to not showing the Vim mode indicator on the screen?

Thanks,
Canh

@rafaelrinaldi
Copy link
Collaborator Author

Hey @nautilus28, this is happening because you probably have Vim mode enabled by default. To disable it you can either remove fish_vi_key_bindings from your Fish configuration file or run fish_default_key_bindings.

@canhduong28
Copy link

@rafaelrinaldi Many thanks for getting back. What I want is to enable Vim mode by default, but not showing mode indicator on the terminal, is it possible?

@rafaelrinaldi
Copy link
Collaborator Author

This is not possible right now. We will show the indicators for vim mode at all times as of #58. I'm happy to accept a PR to get this in though.

@canhduong28
Copy link

@rafaelrinaldi okay, thanks. I will try to make it possible.

@edouard-lopez
Copy link
Member

@rafaelrinaldi we should add some tests on fish_prompt and pre_prompt (which are a big and ugly functions 😢)

@rafaelrinaldi
Copy link
Collaborator Author

@edouard-lopez I agree. We could do this after the v2 release though?

Many thanks for the hard work and getting the peding issues resolved. I feel like we're pretty much ready for a revamped version release, what do you think?

@edouard-lopez
Copy link
Member

@rafaelrinaldi I'm not comfortable with tests as an after-thought. However, as there is plenty of "legacy" code I don't want to block the release, instead add tests little by little and release more frequently.

@edouard-lopez edouard-lopez mentioned this issue Dec 30, 2018
23 tasks
@edouard-lopez edouard-lopez added this to the v2.0.0 milestone Jan 9, 2019
@edouard-lopez
Copy link
Member

edouard-lopez commented Jan 12, 2019

@rafaelrinaldi @schrodincat

I think we should postpone async support to a future version and release a 2.0.0 as we do have a bunch of new features that have been shipped, the code evolved a lot, is stable and tested.
Plus we could announce 3.0.0 support :)

What do you think?

@andreiborisov
Copy link
Collaborator

andreiborisov commented Jan 12, 2019

Seems reasonable to me.

We are not using semantic versioning though?

Also while we are on it, I think we need to make dev branch and commit only release code on the master, since installation scripts are always using latest master commit. Right now user potentially can install some commit in between versions. It’s just safer to have dev branch. I’d even say we can adopt git flow, but that’s not necessary by any means.

@edouard-lopez
Copy link
Member

edouard-lopez commented Jan 12, 2019

@schrodincat cool
Yes we try to follow semver as much as possible, but I introduced some breaking changes on the recent version as I rename some variables in conf.d/pure.fish (74f028d, 731a406, 9f8b8a9).

While revamping the README.md I realized we have some inconsistencies in the naming of variables. This would be nice to improve that before releasing 2.0.0.

@edouard-lopez
Copy link
Member

I wrote a proposal convention for renaming variable in #130 if you want to chime in.

@rafaelrinaldi
Copy link
Collaborator Author

The roadmap looks good to me 👍 Having Git async support behind a feature flag definitely makes sense.

@edouard-lopez edouard-lopez unpinned this issue Feb 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🚀 enhancement performance, UX or maintainability
Projects
None yet
Development

No branches or pull requests

6 participants