Skip to content

return false when file_name cannot convert to str#123

Merged
friedow merged 1 commit intofriedow:mainfrom
prmadev:main
Apr 2, 2024
Merged

return false when file_name cannot convert to str#123
friedow merged 1 commit intofriedow:mainfrom
prmadev:main

Conversation

@prmadev
Copy link
Copy Markdown
Contributor

@prmadev prmadev commented Apr 1, 2024

Since to_str will not fail when the input is ".git" and that input is the only input that matters to us, we can avoid crashing the daemon when to_str returns none.

Since to_str will not fail when the input is ".git" and that input is
the only input that matters to us, we can avoid crashing the daemon when
to_str returns none.

Signed-off-by: Perma Alesheikh <me@prma.dev>
Copy link
Copy Markdown
Owner

@friedow friedow left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for adjusting this @prmadev 👌.

@friedow friedow merged commit 2faf724 into friedow:main Apr 2, 2024
@friedow friedow mentioned this pull request Jun 2, 2024
friedow added a commit that referenced this pull request Jun 2, 2024
## Release Notes

### ✨ New Features
* 💄 display bars to visualize cpu core usage by @friedow in
#121
* ✨ `git_repositories`: add `zoxide` integration by @a-kenji in
#111
* ✨ enable font-fallack for all entries by @friedow in
#131
* 🌈 Allow setting colors via configuration by @pinpox in
#141
* 😜 Gitmoji plugin by @friedow in
#144
* 📑 Firefox bookmarks plugin by @friedow in
#133
* 📜 Firefox History by @friedow in
#152

### 🐛 Fixed Bugs
* 🐛 return false when file_name cannot convert to str by @prmadev in
#123
* 🐛 filter empty plugins for calculating scroll offset by @friedow in
#125
* 🐛 Add sort function to plugin trait by @a-kenji in
#124
* 🐛 implement full desktop spec by @friedow in
#120
* 🐛 fix initial sorting in a bunch of plugins by @friedow in
#129
* 🐛: fix suspend command invocation by @a-kenji in
#134

### Maintenance
* 📝 clarify wayland support by @a-kenji in
#105
* 👷 remove magic-nix-cache action by @a-kenji in
#106
* ✏️ Fix a small typo in the Readme by @a-kenji in
#112
* 🚨 Implement various clippy suggestions by @a-kenji in
#113

### New Contributors
* @prmadev made their first contribution in
#123
* @pinpox made their first contribution in
#141

**Full Changelog**:
v1.0.0...v1.1.0
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