Skip to content

Conversation

@Net-burst
Copy link
Collaborator

@Net-burst Net-burst commented Mar 19, 2025

🔧 Type of changes

  • new bid adapter
  • bid adapter update
  • new feature
  • new analytics adapter
  • new module
  • module update
  • bugfix
  • documentation
  • configuration
  • dependency update
  • tech debt (test coverage, refactorings, etc.)

✨ What's the context?

Bumping the Docker image version to use the newer base image as the current default one will be EOL next year.

🧠 Rationale behind the change

Never image is needed to support some of the dependency update work

🔎 New Bid Adapter Checklist

  • verify email contact works
  • NO fully dynamic hostnames
  • geographic host parameters are NOT required
  • direct use of HTTP is prohibited - implement an existing Bidder interface that will do all the job
  • if the ORTB is just forwarded to the endpoint, use the generic adapter - define the new adapter as the alias of the generic adapter
  • cover an adapter configuration with an integration test

🧪 Test plan

Existing test suite should be enough

🏎 Quality check

  • Are your changes following our code style guidelines?
  • Are there any breaking changes in your code?
  • Does your test coverage exceed 90%?
  • Are there any erroneous console logs, debuggers or leftover code in your changes?

@Net-burst Net-burst requested a review from osulzhenko March 19, 2025 20:05
@Net-burst Net-burst changed the title Bump Docker image version Docker: Update base Docker image version Mar 19, 2025
@osulzhenko osulzhenko added dependencies Pull requests that update a dependency file do not port labels Mar 19, 2025
@Net-burst Net-burst merged commit 68aea26 into master Mar 19, 2025
7 of 8 checks passed
@Net-burst Net-burst deleted the housekeeping/docker-image-bump branch March 19, 2025 23:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file do not port

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants