Skip to content

Update outdated README instructions#18

Open
jatkinson1000 wants to merge 4 commits intolarson-group:masterfrom
m2lines:readme-obtaining
Open

Update outdated README instructions#18
jatkinson1000 wants to merge 4 commits intolarson-group:masterfrom
m2lines:readme-obtaining

Conversation

@jatkinson1000
Copy link

Hi There,

I am looking at CLUBB and am in the process of setting things up.

Whilst following the, generally very comprehensive, README I found that there were some outdated/incorrect instructions around obtaining the code.

Looking through git blame it seems these lines have not been updated in 7 or 9 years.
I presume at this time CLUBB was closed source, requiring login access to a private GitHub repository, but has since been released open source.

I have made changes to reflect this in the README text in this PR.

I also corrected a small typo in the clone command to reflect the text that describes cloning to a repository named differently to the GitHub repo.

@jatkinson1000
Copy link
Author

I note from chapter 6 of the README that you suggest providing patchfiles for contribution.
If this is your preference, rather than merging on GitHub, I can do this, though the changes are relatively minor as viewed in the diff here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants