pluto icon indicating copy to clipboard operation
pluto copied to clipboard

Code cleanup and linter fixes

Open bmos opened this issue 1 year ago • 3 comments

I ran the code through rubocop and ignoring just the line length warnings. You can keep this or leave it, I know code style can be very subjective.

It depends on my other commit #45 and would be easiest to merge if you first merge #45 via rebase and merge.

EDIT: Also the frozen string literal feature at the top of the files doesn't work for at least one file.

bmos avatar Feb 28 '24 21:02 bmos

@bmos thanks for taking the time and the effort. sorry about rubocop i am with the japenese rubyists to quote:

We don't use RuboCop because we can manage our coding style by ourselves. We want to accept small fluctuations in our coding style because we use Ruby. Please do not submit issues and PRs that aim to introduce RuboCop in this repository.

my fault. i will add a note to the readme. happy to see a lint list in text (with warnings) to commit. i am sure there are some good parts in there and again thanks for taking the time. sorry for now my time is unfortunately limited to pick out the good ones. more than happy to work on pluto but again real life realities for now are different.

geraldb avatar Feb 29 '24 09:02 geraldb

That's ok, I made sure to commit anything important to the first PR 🙂

If you're considering README changes, it would be great to also have a CONTRIBUTING.md file with instructions to get the build environment set up and any norms you want people to follow (like not linting existing code). It took me a while to figure some of that stuff out.

bmos avatar Feb 29 '24 11:02 bmos

Greetings from Austria (near Vienna). Thanks for the commentary. The main idea to make contributing easier was to split the "monolith" into gems ("modul-mania") - the idea being only work / contribute to a single gem. Guess a big fail. might be more confusing. and lately i am using the monotree/repo approach / style, thus, lots of additional (experimental / optional) gems here leading to more confusion.

Anyways, again great to hear that pluto is still getting used & found (on the internets). again i try to set aside a couple of days in march 2024 (this upcoming month) to update & housekeeping. thanks for your patience.

geraldb avatar Feb 29 '24 12:02 geraldb