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

cli: add option to choose color manually #407

Closed
wants to merge 3 commits into from

Conversation

pleshevskiy
Copy link
Contributor

@pleshevskiy pleshevskiy commented Sep 3, 2022

I'm currently working on a full integration with fzf and himalaya auto-detects FZF_DEFAULT_COMMAND and --preview as no tty. It would be great if these changes could be merged.

@pleshevskiy
Copy link
Contributor Author

I also fixed the tests, cause someone broke it :)

@soywod
Copy link
Member

soywod commented Sep 10, 2022

Thanks a lot for your contribution 🙂 the v0.6 should come very soon (within few days) with big structural and API changes, I come back to you whenever the development branch is up-to-date!

@soywod
Copy link
Member

soywod commented Oct 10, 2022

I just released the v0.6.0, which includes the code splitting. Sorry for the conflicts! But they should be easy to fix, everything moved down a level (there is no more cli folder). Let me know if you need help. Once the conflicts resolved we will be able to merge your PR 🙂

@soywod soywod changed the base branch from master to develop October 10, 2022 21:01
@soywod
Copy link
Member

soywod commented Dec 15, 2022

I totally forgot about your PR, sorry! I will fix conflicts and merge it straight after #433.

@pleshevskiy
Copy link
Contributor Author

Thanks! I don't have time, sorry

soywod pushed a commit that referenced this pull request Dec 17, 2022
@soywod
Copy link
Member

soywod commented Dec 17, 2022

I was not able to push directly on your fork, so I pushed your changes directly on develop. Thanks a lot for your contribution and sorry for the delay! I should be available in the next release #433.

@soywod soywod closed this Dec 17, 2022
soywod added a commit that referenced this pull request Feb 8, 2023
* update codebase with email lib changes (#431)

update himalaya-lib, rename remaining mbox vars

add missing methods from lib

update changelog

* fixed missing folder aliases #430

* improve README links

* fix README repology link

* fix README repology table

* fix README repology table 2

* center README repology table

* fix README cosmetic issues

* fix README cosmetic issues 2

* fix README title

* fix README wiki links

* fix lock file

* prepare v0.6.2

* fix ci

* try some musl builds #356

* add musl build to artifact #356

* add musl build to deployment pipeline #356

* migrate clap v4, add man command #419

* add option to choose color manually #407

* update links and badges

* update matrix badge

* add github release version badge

* update badges links

* fix code bloc type

* fix tests

* fix cargo lock

* generate all man pages for all subcommands #419

* fix query and headers arg parsers

* fix invalid flags and options due to clap v4 migration

* fix tests

* remove -l|--log-level option

* refactor contributing guide

* update lib

* fix flags string printer

* make commands read, attachments, copy, move and delete accept multiple ids

* fix ids arg parser

* fix flags subcommands conflicts between ids and flags

* flip back copy and move arguments

* add issue template (#439)

* update lib, prepare for sync feature

* update himalaya lib, fix senders and config

* update lock file himalaya lib

* fix sync enabling issues

* fix wrong imap backend init in main file

* fix notmuch backend post sync feature

* configuration wizard (#432)

* make DeserializedConfig::path more robust

With this change, himalaya uses the crate `dirs` in order to follow XDG
specifications on Unix, Known Folder on Windows and Standard Directories
on MacOS. This gives us much smoother cross-platform support. It still
has the same fallbacks (`$HOME/.config/himalaya/config.toml` and
`$HOME/.himalayarc`.)

Additionally, this commit removes a bit of in-house code-bloat.

* add wizard entrypoint and basic structure

* wip

* feat: impl Serialize for all DeserializedConfigs

* feat: select default account and write to file

* feat: add SMTP part of wizard

* build: update lockfile

* refactor: separate out multiple files for wizard

* style: friendlier and prettier messages

* feat: add maildir part of wizard

* feat: add notmuch part of wizard

* chore: clippy lints and reorder prompts

* fix: contrived solution to serializing None values

* fix: allow empty Option field when deserializing

* style: address PR review comments

* fix: utilize notmuch lib in finding database path

* fix notmuch wizard

---------

Co-authored-by: Clément DOUIN <clement.douin@posteo.net>

* add account sync progress bar

* improve sync spinner

* make the sync dry run flag show patches without applying them

* update himalaya lib, increase imap session pool size

* add disable cache flag

* add nlnet logo in readme

* update himalaya lib deps, make use of sync reports

* prepare v0.7.0

* bump rustc v1.67.0 and clap v4.1.4

* bump himalaya lib v0.5.1, fix flake lock file

---------

Co-authored-by: janabhumi <dmitriy@ideascup.me>
Co-authored-by: Knut Magnus Aasrud <km@aasrud.com>
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

Successfully merging this pull request may close these issues.

2 participants