Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Prevent Spotlight from indexing terminal-notifier.app #238

Merged
merged 2 commits into from
Jan 24, 2019
Merged

Prevent Spotlight from indexing terminal-notifier.app #238

merged 2 commits into from
Jan 24, 2019

Conversation

sonicdoe
Copy link
Collaborator

This commit places terminal-notifier.app in a mac.noindex folder so that Spotlight does not index the app (Spotlight does not index folders ending with .noindex).

If you happen to have a few npm packages depending on node-notifier installed, you’ll encounter quite a few terminal-notifier.app results when searching for “Terminal” in Spotlight (or in any other app launcher that uses Spotlight’s index). Since there’s no point in opening terminal-notifier.app by itself and it just clutters Spotlight’s results, I’d suggest removing it from the index.

Side note: Why not .metadata_never_index?

The above-linked Stack Overflow answer also mentions .metadata_never_index as a way to prevent Spotlight from indexing a folder. However, as noted by Nate in the comments and mentioned in OS X Incident Response: Scripting and Analysis, it only seems to work at the root of a volume.

@shnhrrsn
Copy link

shnhrrsn commented Oct 1, 2018

Any movement on this? Was really quite confused why doing cmd+space + ter + enter quickly was causing notifications and not opening terminal 😂

@drewcovi
Copy link

drewcovi commented Jan 9, 2019

@mikaelbr any updates on this? This effects literally every Mac user, and we love node-notifier, but it's turning into a huge headache.

@mikaelbr
Copy link
Owner

Hi, sorry about taking so long! @sonicdoe If you could update the PR I'll merge it right away

According to https://apple.stackexchange.com/a/92785, Spotlight does not index folders ending with `.noindex`.
@sonicdoe
Copy link
Collaborator Author

No worries, thanks for getting back to this. I’ve rebased my pull request now.

@mikaelbr mikaelbr merged commit 85b7eb3 into mikaelbr:master Jan 24, 2019
@sonicdoe sonicdoe deleted the terminal-notifier-spotlight branch January 24, 2019 18:21
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants