Skip to content

Fix configuration keyword mismatch; update alfred.py#3

Open
deanishe wants to merge 4 commits intomwaterfall:masterfrom
deanishe:master
Open

Fix configuration keyword mismatch; update alfred.py#3
deanishe wants to merge 4 commits intomwaterfall:masterfrom
deanishe:master

Conversation

@deanishe
Copy link
Copy Markdown

@deanishe deanishe commented Apr 7, 2014

  • Add latest version of alfred.py to fix Unicode normalisation problems
  • Enable workflow to manage its own configuration (editing JSON is tricky and TextEdit breaks it by default due to Smart Quotes)
  • Add proper argument parsing with argparse
  • Replace duplicated functionality with alfred.py features
  • Use more Pythonic idioms where appropriate
  • Remove *.pyc files from repo
  • Add .gitignore to keep *.pyc files out of source control
  • Remove unnecessary __init__.py
  • Rebuild distributable workflow file
  • Use icon of file in Alfred's results, not workflow icon

deanishe added 4 commits April 7, 2014 12:17
- Add latest version of alfred.py to fix Unicode normalisation problems
- Replace duplicated functionality with alfred.py features
- Use more Pythonic idioms where appropriate
- Increase number of results sent to Alfred from 9 to 50
- Remove `exit` call in library to give control to calling code
- Fix config/setup keyword mix-up between code and Run Script action
- Remove *.pyc files from repo
- Add .gitignore to keep *.pyc files out of source control
- Remove TextEdit as editor for JSON config

By default, TextEdit will break the JSON by using "Smart Quotes"
The config file should be opened with the user's default editor
for JSON files, which will presumably do a better job than TextEdit, and
can hardly be worse.
- Add File Action to add folders
- Add `qrfolders` keyword to display folders
- Add result action to remove folders
- Remove outdated screenshot
- Add usage info to README
@deanishe
Copy link
Copy Markdown
Author

Any chance you're going to accept this pull request?

Otherwise, I'll fork the workflow and distribute it separately.

@mwaterfall
Copy link
Copy Markdown
Owner

Sorry, looks like this is "unmaintained" at the moment lol. I'm extremely busy at the moment, so feel free to fork it!

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