Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Bats repository #46

Merged
merged 1 commit into from
Jan 26, 2019
Merged

Update Bats repository #46

merged 1 commit into from
Jan 26, 2019

Conversation

scontini76
Copy link
Contributor

the current repository isn't maintained anymore, Bats development is going forward on https://github.com/bats-core/bats-core

the current repository isn't maintained anymore, Bats development is going forward on https://github.com/bats-core/bats-core
@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.048% when pulling 58ec85a on scontini76:patch-1 into 9462f7d on infertux:master.

@tomeon
Copy link
Collaborator

tomeon commented Jan 25, 2019

@scontini76 -- this LGTM. @infertux -- want to leave a link to the old Bats repo for posterity, or does it suffice that the bats-core repo itself links to the original sstephenson repo?

@infertux infertux merged commit 4ffb042 into infertux:master Jan 26, 2019
@infertux
Copy link
Owner

@scontini76 Good catch, thank you!

does it suffice that the bats-core repo itself links to the original sstephenson repo

@tomeon I think it does.

@scontini76 scontini76 deleted the patch-1 branch January 26, 2019 09:02
@scontini76
Copy link
Contributor Author

You are welcome!

The only clear evidence about the past of bats-core is the copyright to Sam Stephenson. On the other side the https://github.com/sstephenson/bats/ is really abandoned and I see many that opens bugs and pulls request without success (ex. sstephenson/bats#262, I had the same problem in the past).

So, even if there are no links to the original repository I think that is better for users to don't be confused by old useless link, just my 2c!

Thank you for the merge!

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.

4 participants