Skip to content

Conversation

@krzysztofequativ
Copy link

No description provided.

@github-actions
Copy link

Whoa there, partner! 🌵🤠 We wrangled some duplicated code in your PR:

Reducing code duplication by importing common functions from a library not only makes our code cleaner but also easier to maintain. Please move the common code from both files into a library and import it in each. We hate that we have to mention this, however, commits designed to hide from this utility by renaming variables or reordering an object are poor conduct. We will not look upon them kindly! Keep up the great work! 🚀

@github-actions
Copy link

Tread carefully! This PR adds 1 linter error (possibly disabled through directives):

  • modules/equativBidAdapter.js (+1 error)

@krzysztofequativ krzysztofequativ merged commit e852c96 into master Apr 24, 2025
4 of 6 checks passed
@krzysztofequativ krzysztofequativ deleted the min-bid-to-win branch April 24, 2025 15:36
krzysztofequativ pushed a commit that referenced this pull request Nov 19, 2025
* AGT685: Remove device info logic (#42)

* AGT-680: Don't send gam in browserBlacklist (#44)

* AGT-680: Dont send gam in browserBlacklist

* AGT-680: fix commas

* Agt 682 implementation for module (#45)

* add GAM prediction

* Update version number, add tests, update documentation

* Fix property name

* refactor export file

* fix import typo

* remove GAM prediction  flag

* extend GAM prediction documentation

* fix configuration name in documentation

* Agt 698 module config changes (#46)

* move analytic related params to analytic adapter

* Move back Maintainer section

---------

Co-authored-by: dmytro-po <dmytro.po@intentiq.com>
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.

2 participants