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

Freewheel: add video dur and cat #3640

Merged
merged 5 commits into from
May 2, 2024

Conversation

mwang-sticky
Copy link
Contributor

No description provided.

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 7ecef76

freewheelssp

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:19:	MakeRequests	83.3%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:64:	MakeBids	95.2%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:109:	Builder		100.0%
total:										(statements)	90.2%

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, d0f5acf

freewheelssp

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:19:	MakeRequests	83.3%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:64:	MakeBids	95.2%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:109:	Builder		100.0%
total:										(statements)	90.2%

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 098262d

freewheelssp

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:19:	MakeRequests	83.3%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:64:	MakeBids	95.2%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:109:	Builder		100.0%
total:										(statements)	90.2%

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, c385343

freewheelssp

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:19:	MakeRequests	83.3%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:64:	MakeBids	95.2%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:109:	Builder		100.0%
total:										(statements)	90.2%

@bsardo bsardo changed the title freewheel-adapter: add video dur and cat Freewheel: add video dur and cat Apr 25, 2024
Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, a26a598

freewheelssp

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:19:	MakeRequests	83.3%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:64:	MakeBids	95.2%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:109:	Builder		100.0%
total:										(statements)	90.2%

@gargcreation1992
Copy link
Contributor

@mwang-sticky PR checks are failing because of - expected.ImpIDs must contain at least one imp ID

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 37db0df

freewheelssp

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:19:	MakeRequests	83.3%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:64:	MakeBids	95.2%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:109:	Builder		100.0%
total:										(statements)	90.2%

@mwang-sticky
Copy link
Contributor Author

@mwang-sticky PR checks are failing because of - expected.ImpIDs must contain at least one imp ID

30082c0#diff-d83f728c5918299ec6f5a8d22447adb9fefac2726ab505dea12ba609dd79e068 ?

Copy link

Code coverage summary

Note:

  • Prebid team doesn't anticipate tests covering code paths that might result in marshal and unmarshal errors
  • Coverage summary encompasses all commits leading up to the latest one, 9924f99

freewheelssp

Refer here for heat map coverage report

github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:19:	MakeRequests	83.3%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:64:	MakeBids	95.2%
github.com/prebid/prebid-server/v2/adapters/freewheelssp/freewheelssp.go:109:	Builder		100.0%
total:										(statements)	90.2%

@Sonali-More-Xandr Sonali-More-Xandr merged commit 6520bac into prebid:master May 2, 2024
5 checks passed
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.

3 participants