Skip to content

Conversation

@chrfwow
Copy link
Contributor

@chrfwow chrfwow commented Nov 27, 2025

A crude POC to add a test that will fail when a breaking change is introduced

DO NOT MERGE!!!!!!!

Signed-off-by: christian.lutnik <christian.lutnik@dynatrace.com>
Signed-off-by: christian.lutnik <christian.lutnik@dynatrace.com>
Signed-off-by: christian.lutnik <christian.lutnik@dynatrace.com>
Signed-off-by: christian.lutnik <christian.lutnik@dynatrace.com>
* @param details The details of the event
*/
public Awaitable emitProviderReady(ProviderEventDetails details) {
/*public*/ Awaitable emitProviderReady(ProviderEventDetails details) {
Copy link
Contributor Author

@chrfwow chrfwow Nov 27, 2025

Choose a reason for hiding this comment

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

Suggested change
/*public*/ Awaitable emitProviderReady(ProviderEventDetails details) {
public Awaitable emitProviderReady(ProviderEventDetails details) {

Revert. This is my breaking change which I want to catch with this test

Signed-off-by: christian.lutnik <christian.lutnik@dynatrace.com>
Signed-off-by: christian.lutnik <christian.lutnik@dynatrace.com>
Signed-off-by: christian.lutnik <christian.lutnik@dynatrace.com>
Signed-off-by: christian.lutnik <christian.lutnik@dynatrace.com>
Signed-off-by: christian.lutnik <christian.lutnik@dynatrace.com>
@chrfwow chrfwow changed the title Test for breaking changes test: Test for breaking changes Nov 27, 2025
Signed-off-by: christian.lutnik <christian.lutnik@dynatrace.com>
Signed-off-by: christian.lutnik <christian.lutnik@dynatrace.com>
@sonarqubecloud
Copy link

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.

1 participant