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

Improve language in README.md #16

Merged
merged 4 commits into from
May 20, 2021
Merged

Improve language in README.md #16

merged 4 commits into from
May 20, 2021

Conversation

azadad96
Copy link
Contributor

Edited some of the README.md to fix typos or reformulate the sentences.
It's not perfect yet, since I wasn't always completely sure what was meant by the original author.

Copy link

@biru-cs biru-cs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You fixed a lot of typos and grammar errors, but I also improved a couple of lines. Nice job :)

README.md Outdated
### Windows(not tested)
Packed zip file contains all dependences, but if you want, you can install https://github.com/tschoonj/GTK-for-Windows-Runtime-Environment-Installer/releases and move file whatever you want.
### Windows (not tested)
Packed zip file contains all dependencies, but if you want to, you can install https://github.com/tschoonj/GTK-for-Windows-Runtime-Environment-Installer/releases and move file whatever you want.
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

...if you want to, you can install the GTK runtime from (link)...

README.md Outdated
@@ -56,29 +56,30 @@ cargo install szyszka
## Future work
- Adding Regex support
- New icon
- Allow to add folders
- Allow adding folders
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Allow folder addition

@qarmin
Copy link
Owner

qarmin commented May 16, 2021

Needs rebase after recent changes

@MGlolenstine MGlolenstine mentioned this pull request May 16, 2021
@qarmin qarmin merged commit b1154f7 into qarmin:master May 20, 2021
@qarmin
Copy link
Owner

qarmin commented May 20, 2021

Thanks!

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.

None yet

3 participants