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.

Sign up for GitHub

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

Jump to bottom

Code commenting #31

Open
Efemache opened this issue Nov 15, 2021 · 1 comment
Open

Code commenting #31

Efemache opened this issue Nov 15, 2021 · 1 comment

Comments

@Efemache
Copy link
Contributor

Hi,

Could you put comments in your code please ?

Yesterday, I tried to change a part of the bot and I read ~1h30 to understand the code, functions, what the bot do, where it clicks and why, ... to find the only line I had to change.

When you, finally, understand each functions, returns, etc.... it's easy to find what you have to look for (and where) to change something, but when you don't know it, the code is very hard to understand.
@Efemache
Copy link
Contributor Author

I started to work on it

Efemache referenced this issue in Efemache/Mercenaries-Farm-bot Nov 16, 2021
@Efemache
little bit of documentation added for tab 'buttons', 'ui', 'chekers' …
54a8f15 
…and 'setings' (issues #31)
@Efemache Efemache mentioned this issue Nov 16, 2021
Merged
Deopster pushed a commit that referenced this issue Nov 17, 2021
@Efemache
A lot of improvements for Linux, window support, bug fix, (#37)
5c9d8f2 
* little bit of documentation added for tab 'buttons', 'ui', 'chekers' and 'setings' (issues #31)

* Heroes card selection (to put on board) improved and debug mod added

* Select a correct enemy when there is an odd number of enemies

Image with Russian text replaced with the same image with English text

* Window experience added (you can use fullscreen or window)

You need to use of the supported resolution

* Information added about prerequesite (AutoHotKey)

* file renamed for Linux (case sensitive)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@Efemache