search

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 32 forks source link

Need sudo in README.md command #323

Closed timur-harin closed 3 months ago

timur-harin commented 3 months ago

@yegor256

In section in README.md

How to Contribute

We have these commands

sudo make install make test lint

But then run second command, user can have this

before/test-executability.sh: rm: cannot remove '/home/user/develop/cam/test-zone/before/test-executability.sh/target': Permission denied

So I suggest to change command with sudo, because in testing we have deletion with the inaccessible level:

sudo make install sudo make test lint

not

sudo make install test lint

because all commands in one line can confuse

timur-harin commented 3 months ago

@yegor256 Please check the solution to the problem PR #324

timur-harin commented 3 months ago

@yegor256 Close this issue because PR #324 was merged