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

Is it a common practice to make one commit mixing the changes with origin codes, when forked other's repository? #6

Closed
PsiACE opened this issue Jan 24, 2022 · 3 comments

Comments

@PsiACE
Copy link
Owner

PsiACE commented Jan 24, 2022

ref: databendlabs/databend#3979

Hi @wangzhen11aaa , this question is about ritelinked, and I think we can open an issue at https://github.com/dataslime/ritelinked .


I'm glad you're interested in ritelinked. ritelinked is derived from hashlink and they share most of the code logic. The main differences are these:

  • some new features, such as amortized, support no_std
  • better maintenance and well organization (eg. ritelinked's serde module was fixed earlier than hashlink)
  • some functions such as get_refresh, retain

see old commit history: https://github.com/dataslime/ritelinked/commits/ritelabs/ritelinked

Also, it is a fact that ritelinked is pretty much my personal hashlink fork, although it is used by some projects. Since I plan to do some rewrites, I overwrite the commit records under the main branch, sorry for the confusion.

@PsiACE
Copy link
Owner Author

PsiACE commented Jan 24, 2022

I admit there are some problems here, such as @kyren and others' commits being ignored, and I simply pointed out in the readme that this is a fork of hashlink. I'm sorry I didn't take care of their feelings at the time, maybe I should have kept their names in the program.

I have tried to send these changes back upstream and turn off ritelinked, but have not gotten feedback. kyren/hashlink#9

For the time being, I might consider keeping ritelinked and spending more time rewriting it to be a competitor to hashlink.

@PsiACE
Copy link
Owner Author

PsiACE commented Jan 24, 2022

So back to the question, I don't think the behavior of ritelinked is a common practice.

@wangzhen11aaa
Copy link

That's Excellent

@PsiACE PsiACE closed this as completed Jan 25, 2022
@PsiACE PsiACE pinned this issue Jan 25, 2022
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

No branches or pull requests

2 participants