• Joined on 2024-02-19
yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

should be const

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

Please clean up

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

Please clean up

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

variables for these string values would be nice

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

It would be helpful to mention other possible statuses

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

it would be nice to define response type

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

one-line comment would make more sense

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

should be const

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

should be a const

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

please clean up

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

should be const

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

should be const

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

should be const

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

Please clean up

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

lint check should throw a warning

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

should be const

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

one-line comment would make more sense

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

should be const

yury commented on pull request dc/dc#115 2024-06-27 07:38:39 +00:00
feat(multi load): refactored range find function, unlocking excel with password is reusable

should be const