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

Add a test for the HTTP service bug classification with missing bugs #3358

Closed
wants to merge 8 commits into from

Conversation

Emeka-Onwuepe
Copy link
Contributor

@Emeka-Onwuepe Emeka-Onwuepe commented Mar 15, 2023

please verify if I am doing the right thing
fixes #1368

@Emeka-Onwuepe
Copy link
Contributor Author

The code is in test for Bugzilla API in test_model_predict_batch

i need to remove
Refactored classify_bug func in bugbug_http/models
I am sorry, I fogort

@suhaibmujahid
Copy link
Member

Thank you, @Emeka-Onwuepe! Could you please link this PR to the issue that it is fixing? Also, if this is still a work in progress, please convert it to a draft pull request.

@Emeka-Onwuepe
Copy link
Contributor Author

Ok sir, fixes #1368

@suhaibmujahid
Copy link
Member

Ok sir, fixes #1368

The linking keyword should be in the description; see https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue

@suhaibmujahid
Copy link
Member

Ok sir, fixes #1368

It seems that issue #1368 already has an open pull request linked to it. Please read #1092.

@Emeka-Onwuepe
Copy link
Contributor Author

Ok sir, fixes #1368

It seems that issue #1368 already has an open pull request linked to it. Please read #1092.

Please look at the poll requests time, I already made a pull request before an empty code was pushed and linked by another.

@Emeka-Onwuepe
Copy link
Contributor Author

Ok sir, fixes #1368

The linking keyword should be in the description; see https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue

Ok sir, I am going through the write up, thanks

@Emeka-Onwuepe
Copy link
Contributor Author

@suhaibmujahid should I continue working on this or should I close it?

@suhaibmujahid
Copy link
Member

Please look at the poll requests time, I already made a pull request before an empty code was pushed and linked by another.

But the PR was not linked to the issue and still until now not linked to the issue.

Ok sir, I am going through the write up, thanks

You just need to edit the description to add the following:

fixes #1368

@suhaibmujahid should I continue working on this or should I close it?

Feel free to continue working on it, but please link it to the issue.

@Emeka-Onwuepe
Copy link
Contributor Author

Emeka-Onwuepe commented Mar 15, 2023

Please look at the poll requests time, I already made a pull request before an empty code was pushed and linked by another.

But the PR was not linked to the issue and still until now not linked to the issue.

Ok sir, I am going through the write up, thanks

You just need to edit the description to add the following:

fixes #1368

@suhaibmujahid should I continue working on this or should I close it?

Feel free to continue working on it, but please link it to the issue.

Thanks sir for the assistance, I believe I have been able to link it now.

Please, look at test for Bugzilla API in test_model_predict_batch to know if I am on track.

@suhaibmujahid
Copy link
Member

Thanks sir for the assistance,

It is my pleasure.

I believe I have been able to link it now.

Thank you.

Please, look at test for Bugzilla API in test_model_predict_batch to know if I am on track.

The CI is failing, please fix that :)

@Emeka-Onwuepe
Copy link
Contributor Author

Ok sir

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.

Add a test for the HTTP service bug classification with missing bugs
2 participants