Skip to content

Conversation

@bretg
Copy link
Contributor

@bretg bretg commented Jan 8, 2025

This is mostly to test the codepath notification

🔧 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?

Mainly testing the codepath alerts.

This is mostly to test the codepath notification
@@ -1,5 +1,6 @@
adapters:
ix:
# Please contact the maintainer to obtain endpoint details
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To confirm - Index received email from Prebid Infoinfo@prebid.org for this PR. Notification is working effectively! 🎉

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question: are we doing same for PBS-Go?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are we doing same for PBS-Go?

Now that we've tested it in this repo, I'll open a PR to get the same behavior in the PBS-Go repo.

Copy link
Contributor

@oronno oronno left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@CTMBNara CTMBNara merged commit f0c65af into master Jan 17, 2025
8 checks passed
@CTMBNara CTMBNara deleted the codepath-test branch January 17, 2025 13:31
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.

4 participants