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

Use bulk endpoint for scanning #170

Closed
wants to merge 49 commits into from
Closed

Conversation

tuxinal
Copy link

@tuxinal tuxinal commented Jul 11, 2022

No description provided.

@CLAassistant
Copy link

CLAassistant commented Jul 11, 2022

CLA assistant check
All committers have signed the CLA.

@tuxinal
Copy link
Author

tuxinal commented Jul 11, 2022

i appear to have done a poor job at merging upstream sorry about that (mainly towards sidit77 as this does not involve you at all)

@theofficialgman
Copy link

theofficialgman commented Aug 24, 2022

just a thought... you shouldn't have been using git merge in the first place.

if you were working on your local tree and it was out of date compared to upstream, you should have done a git interactive rebase on the upstream branch. that would apply your commits on top of the upstream branch and would prompt you with any conflicts which you might need to fix

with the mess you have now, it would be best to just just apply one commit on top of upstream

@huantianad
Copy link

Yeah giving up and squashing things is probably the way to go :P. If you have time you could probably spend some time rebasing and splitting things up sensibly, that would be the best thing to do, but that'll probably take a bit

@tuxinal tuxinal closed this Sep 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants