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

Refactor single file use and add tooling #26

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

pythoninthegrass
Copy link

Hi @VikParuchuri!

Thanks for making this! Added some general quality-of-life features/docs and light edits to the convert_single.py script to get it in line with python conventions (e.g., main() vs magic/dunder method call).

Probably missed some stuff in subdirectories and won't be offended if you only use some of the below.

CHANGELOG

  • all scripts in top-level directory: chmod +x
  • convert_single.py: shebang, sort imports (comment out unused settings import), move argparse to global scope, break out main function from magic method call
  • .gitignore: add Brewfile.lock.json exclusion
  • .vscode: configure python debugging w/boilerplate args, general python editor settings
  • README.md: replace generic brew requirements file w/semantic Brewfile and instructions; minimal shell scripting to get tessdata directory on mac; auto newline at EOF (unix requirement)
  • Brewfile: requirements file to programmatically install brew formulae

- all scripts in top-level directory: `chmod +x`
- `convert_single.py`: shebang, sort imports (comment out unused settings import), move argparse to global scope, break out main function from magic method call
- `.gitignore`: add `Brewfile.lock.json` exclusion
- `.vscode`: configure python debugging w/boilerplate args, general python editor settings
- `README.md`: replace generic brew requirements file w/semantic `Brewfile` and instructions; minimal shell scripting to get `tessdata` directory on mac; auto newline at EOF (unix requirement)
- `Brewfile`: requirements file to programmatically install brew formulae
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.

1 participant