-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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 inventory-notifier #7520
base: master
Are you sure you want to change the base?
Conversation
We would prefer you to keep a linear history, it's going to take longer to review if you force push your changes and we have to review the entire plugin, again. |
Yeah ^ please rework your commits to have a linear history since there's no need here for a new git timeline. |
Hi, I’m not that proficient with Git, and deleting the old branch was suggested as a fix for the errors I was running into—it seems to have resolved them. I’m not sure how I’d go about restoring old history. That said, this is a pretty small/basic plugin, and almost everything was rewritten, so I don’t think much of the original code remains. Please let me know if there’s anything specific you’d like me to do. Thanks! |
You might be able to recover using a commit you used in one of the other PRs you closed (which you don't need to do btw, just update your existing PR). You might also be able to find an older state to reset to using |
Updated for current RuneLite API.