Skip to content

chore: metrics collection job should also run on develop #6882

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

Merged
merged 1 commit into from
Feb 8, 2023

Conversation

vaind
Copy link
Contributor

@vaind vaind commented Jan 19, 2023

The previous PR that introduces the Replay SDK metrics job already contained code that collected baseline info from the main branch.
However, after making a change to only run when a specific label is added, this feature got lost...
This PR should fix that.

@vaind vaind force-pushed the chore/metrics-main-branch branch 2 times, most recently from 33c5ce8 to 7e16123 Compare January 20, 2023 09:35
@vaind vaind marked this pull request as ready for review January 20, 2023 09:35
Lms24
Lms24 previously approved these changes Feb 1, 2023
@mydea mydea changed the base branch from master to develop February 2, 2023 09:46
@mydea mydea dismissed Lms24’s stale review February 2, 2023 09:46

The base branch was changed.

@mydea
Copy link
Member

mydea commented Feb 2, 2023

In the meanwhile we changed the main branch to develop (I updated the PR target). please rebase this on latest develop, and change is_master to is_develop :)

@vaind vaind force-pushed the chore/metrics-main-branch branch from 7e16123 to caf4a03 Compare February 2, 2023 19:48
@vaind vaind force-pushed the chore/metrics-main-branch branch from caf4a03 to 4a7cd8e Compare February 2, 2023 19:49
@vaind
Copy link
Contributor Author

vaind commented Feb 2, 2023

In the meanwhile we changed the main branch to develop (I updated the PR target). please rebase this on latest develop, and change is_master to is_develop :)

done

@vaind vaind changed the title chore: metrics collection job should also run on master chore: metrics collection job should also run on develop Feb 2, 2023
@Lms24 Lms24 merged commit 6462b00 into getsentry:develop Feb 8, 2023
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.

3 participants