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

Other ideas #5

Merged
merged 52 commits into from
Jan 12, 2019
Merged

Other ideas #5

merged 52 commits into from
Jan 12, 2019

Conversation

doekman
Copy link
Collaborator

@doekman doekman commented Jan 4, 2019

  • Gave the REAMDE some needed TLC
  • Comments now can be aligned (option: comment_align)
  • It's now easier to create your own command based on ok-bash (options: file/alias)
  • It's now easier to reload ok-bash with different options (installation helper: reset)
  • Fixed quite some bugs
  • Replaced awk by python

doekman and others added 30 commits September 24, 2018 15:25
Except the following:
SC2164: Folders will always exist
SC2155: Result code is always 0, or I don't use it at the moment
So `ok -f README.md` now also works correctly
- and renamed "elastic_tab" to "comment_align"
- Added installation helper for comment_align
- Added _OK_COMMENT_ALIGN variable to reset helper
- Fixed some bugs with code lines without comments
- UTF-8 BOM documents work again
doekman and others added 22 commits November 19, 2018 12:55
- and added legal_values to get_env::int
- Fixed crash, when .ok file doesn't contain commands
- Exit with 1 when there are no commands (and in this case don't show prompt when specified)
- changed `which` to `command -v` because shellcheck said so
- made ok available to sub processes via `export -f`
- so `ok` can be used with external scripts (user defined variables need to be exported manually in this case)
- fixed a bug where `tab_stop_step` indented one position to much
- and made `tab_stop_step` defaults to 1 (making it effectly do nothing default)
RIP nice variable name...
will this work? 🤞
@secretGeek secretGeek merged commit f6cf39a into secretGeek:master Jan 12, 2019
@doekman doekman deleted the other-ideas branch January 13, 2019 13:53
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