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

[chore] move googlecloudpubsub exporter to generated lifecycle tests #30525

Conversation

Frapschen
Copy link
Contributor

Relates to #27849

@fatsheep9146 fatsheep9146 added the Skip Changelog PRs that do not require a CHANGELOG.md entry label Jan 15, 2024
@atoulme atoulme force-pushed the googlecloudppubsub_exporter_lifecycle branch from 49fbc73 to 1c584e9 Compare January 31, 2024 21:22
@atoulme
Copy link
Contributor

atoulme commented Jan 31, 2024

@alexvanboxel please review as codeowner, thanks!

Copy link
Contributor

@alexvanboxel alexvanboxel left a comment

Choose a reason for hiding this comment

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

LGTM

@fatsheep9146 fatsheep9146 added the ready to merge Code review completed; ready to merge by maintainers label Feb 1, 2024
@codeboten codeboten removed the ready to merge Code review completed; ready to merge by maintainers label Feb 7, 2024
Copy link
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions bot added the Stale label Feb 22, 2024
@dmitryax
Copy link
Member

@Frapschen can you please rebase?

@atoulme atoulme force-pushed the googlecloudppubsub_exporter_lifecycle branch 2 times, most recently from 3ed952f to f03539b Compare February 28, 2024 21:14
@atoulme atoulme force-pushed the googlecloudppubsub_exporter_lifecycle branch from f03539b to c49fdb8 Compare February 28, 2024 21:15
@atoulme
Copy link
Contributor

atoulme commented Feb 28, 2024

I rebased.

@github-actions github-actions bot removed the Stale label Feb 29, 2024
…cycle' into googlecloudppubsub_exporter_lifecycle

# Conflicts:
#	exporter/googlecloudpubsubexporter/generated_component_test.go
#	exporter/googlecloudpubsubexporter/go.mod
#	exporter/googlecloudpubsubexporter/metadata.yaml
# Conflicts:
#	exporter/googlecloudpubsubexporter/generated_component_test.go
#	exporter/googlecloudpubsubexporter/go.mod
#	exporter/googlecloudpubsubexporter/metadata.yaml
@dmitryax
Copy link
Member

Is it expected to be no changes in the diff?

@atoulme
Copy link
Contributor

atoulme commented Mar 20, 2024

I think this was applied through the generalization of tests then. Closing!

@atoulme atoulme closed this Mar 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exporter/googlecloudpubsub ready to merge Code review completed; ready to merge by maintainers Skip Changelog PRs that do not require a CHANGELOG.md entry waiting-for-code-owners
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants