yegor256 / cam

Classes and Metriсs (CaM): a dataset of Java classes from public open-source GitHub repositories
http://cam.yegor256.com
MIT License
23 stars 38 forks source link

Install reqired `tlmgr` in install script if necessary #215

Closed howcanunot closed 7 months ago

howcanunot commented 7 months ago

tlmgr is part of texlive package. We don't really want to install the full package, so if the utility was not installed before running the script, we try to install it with minimal dependencies, since installing the whole package can take a lot of time and about 6 GB of disk space

Also a bit of code refactoring to remove unnecessary copy-paste

https://github.com/yegor256/cam/issues/174

yegor256 commented 7 months ago

@howcanunot appreciate your contribution! Please, try to run make clean lint locally, to make sure your scripts don't violate quality rules. Also, don't forget that the scripts are multi-platform: they have to work both on MacOS and Ubuntu and CentOS

howcanunot commented 7 months ago

@yegor256 good evening! improved the script to satisfy the multiplatform condition. linter checks is ok

howcanunot commented 7 months ago

@yegor256 Can you please help with debugging the bibcop check? I'm not sure I understand how to fix it... It fails for both my pr's

Or i just should wait for the fix and then pull.

yegor256 commented 7 months ago

@howcanunot thanks!