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

Discovery Bid Adapter :parameter change #9212

Closed
wants to merge 30 commits into from
Closed

Discovery Bid Adapter :parameter change #9212

wants to merge 30 commits into from

Conversation

cuihua111
Copy link
Contributor

@cuihua111 cuihua111 commented Nov 8, 2022

Type of change

  • Bugfix

  • Feature

  • New bidder adapter

  • Code style update (formatting, local variables)

  • Refactoring (no functional changes, no api changes)

  • Build related changes

  • CI related changes

  • Does this change affect user-facing APIs or examples documented on http://prebid.org?

  • Other

Description of change

Other information

@cuihua111 cuihua111 changed the title parameter change discovery bid adapter:parameter change Nov 8, 2022
@cuihua111
Copy link
Contributor Author

cuihua111 commented Nov 9, 2022

I want to know why each commit doesn't automatically trigger circleCI to check?

@SylviaF
Copy link
Contributor

SylviaF commented Nov 15, 2022

Hi @ChrisHuie , I'm one of the dev of Discovery's Prebid Adapter.

We commited a change last week, but it hasn't been merged or received any feedback until this week.
We would like to merge into master as soon as possible, because the publisher won't integrate our adapter before the new version were merged into master and be released.

So would you like to help us to give us some feedback, what do we need to do to make our new change be merge into master and be published in next version?

We need your help so much! Thank you very much for your time!

@ChrisHuie ChrisHuie changed the title discovery bid adapter:parameter change Discovery Bid Adapter :parameter change Nov 15, 2022
@ChrisHuie
Copy link
Collaborator

@SylviaF @cuihua111 I've tried a couple things but looks like I am not able to manually kick off the CircleCI tests and they aren't triggering. I will check this pr out and if I can't get it to trigger I can just test it under my github handle because that is where the CircleCI differs for people.

@ChrisHuie ChrisHuie self-requested a review November 15, 2022 12:56
@ChrisHuie ChrisHuie self-assigned this Nov 15, 2022
@cuihua111
Copy link
Contributor Author

@ChrisHuie Thank you so much, but I want to know if there is a better way to solve the problem of circleCI, otherwise I will trouble you every time I commit

@SylviaF
Copy link
Contributor

SylviaF commented Nov 16, 2022

@SylviaF @cuihua111 I've tried a couple things but looks like I am not able to manually kick off the CircleCI tests and they aren't triggering. I will check this pr out and if I can't get it to trigger I can just test it under my github handle because that is where the CircleCI differs for people.

Hi @ChrisHuie , thanks so much for your effort.

We have recommited a new change and seen that the CircleCI tests is triggerd.
You can jump to the following link to do the remaining work. Thank you!!!
#9249

@ChrisHuie
Copy link
Collaborator

@SylviaF @cuihua111 just merged 👍

FYI. Usually the CircleCI issue comes from someone following their fork of Prebid instead of the main prebid project in the projects section on the side of CircleCI. Unfollowing the fork usually fixes it

@ChrisHuie ChrisHuie closed this Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants