onivim/oni

Allow wildcards in bookmarks

Open

#2.562 geöffnet am 10. Sept. 2018

Auf GitHub ansehen
 (0 Kommentare) (1 Reaktion) (0 zugewiesene Personen)TypeScript (11.389 Stars) (336 Forks)batch import
Feature-requestgood first issueinsider

Beschreibung

Currently, Oni bookmarks are defined as an array like so:

"oni.bookmarks": ["/home/ryan/git/oni", "/home/ryan/git/uni", "/home/ryan/git/dotfiles"...]

What would be nice is some form of wildcard expansion/globbing such that instead a user could do:

"oni.bookmarks": ["/home/ryan/git/*"]

And then add every git project say, or all projects in a folder and so on.

I think this would just be a case of hooking up some logic over in BookmarksSearch.ts to expand the wildcard in the constructor where we grab all bookmarks. There is some discussion of this over in #2554, including a package we could use.

We'd most likely want to restrict this to absolute paths only, so a users home dir isn't used etc.

Contributor Guide

Tech Stack
typescript
Domain
frontendtooling
Issue Type
feature
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
2
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
1-3 hours
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
stale
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
clear
Voraussetzungen
TypeScriptNode.jsglobbing patterns
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
70
Research-Richtung
Start by examining the file 'BookmarksSearch.ts' to understand how bookmarks are currently loaded and stored. Look at the discussion in issue #2554 for suggested packages like 'glob' or 'minimatch'. Implement wildcard expansion in the constructor where bookmarks are gathered, ensuring only absolute paths are expanded.