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

Update Measurement Service Examples to include service Annotations #296

Merged
merged 46 commits into from
Jun 30, 2023

Conversation

gpachecoNI
Copy link
Collaborator

TODO: Check the above box with an 'x' indicating you've read and followed CONTRIBUTING.md.

TODO: Include high-level description of the changes in this pull request.

  • Updates all the existing examples ServiceConfig files.

TODO: Justify why this contribution should be part of the project.

AB#2408764

What testing has been done?

Manually ran all examples.

TODO: Detail what testing has been done to ensure this submission meets requirements.

@gpachecoNI gpachecoNI changed the title Users/gpacheco/update examples Updaty Measurement Service Examples to include service Annotations Jun 23, 2023
@gpachecoNI gpachecoNI requested a review from cumitche June 23, 2023 14:42
@dixonjoel dixonjoel changed the title Updaty Measurement Service Examples to include service Annotations Update Measurement Service Examples to include service Annotations Jun 23, 2023
@gpachecoNI gpachecoNI mentioned this pull request Jun 23, 2023
1 task
@dixonjoel dixonjoel changed the title Update Measurement Service Examples to include service Annotations [Do not submit] Update Measurement Service Examples to include service Annotations Jun 23, 2023
@gpachecoNI gpachecoNI force-pushed the users/gpacheco/AddServiceAnnotationToGenerator branch from dea1d86 to db26e6d Compare June 23, 2023 15:35
@bkeryan
Copy link
Collaborator

bkeryan commented Jun 26, 2023

FYI, I created a releases/1.1 branch and updated main to version 1.2.0-dev0 in order to unblock these PRs.

@gpachecoNI gpachecoNI force-pushed the users/gpacheco/AddServiceAnnotationToGenerator branch from db26e6d to 4cb83ef Compare June 27, 2023 22:40
@gpachecoNI gpachecoNI changed the title [Do not submit] Update Measurement Service Examples to include service Annotations Update Measurement Service Examples to include service Annotations Jun 28, 2023
@gpachecoNI gpachecoNI force-pushed the users/gpacheco/UpdateExamples branch from 0c37b50 to 980fe63 Compare June 28, 2023 15:58
@gpachecoNI gpachecoNI requested a review from cumitche June 28, 2023 16:06
@gpachecoNI gpachecoNI force-pushed the users/gpacheco/AddServiceAnnotationToGenerator branch from 4cb83ef to 62d90ec Compare June 29, 2023 18:36
@gpachecoNI gpachecoNI force-pushed the users/gpacheco/UpdateExamples branch from 1249673 to 74fc3b1 Compare June 29, 2023 18:54
@gpachecoNI gpachecoNI requested a review from bkeryan June 29, 2023 18:55
Copy link
Collaborator

@bkeryan bkeryan left a comment

Choose a reason for hiding this comment

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

Approved with suggestions

@gpachecoNI gpachecoNI force-pushed the users/gpacheco/UpdateExamples branch from 74fc3b1 to 24e45ac Compare June 29, 2023 22:23
Base automatically changed from users/gpacheco/AddServiceAnnotationToGenerator to main June 30, 2023 16:47
@github-actions
Copy link

Test Results

     12 files  ±0       12 suites  ±0   16m 22s ⏱️ -45s
   174 tests ±0     174 ✔️ ±0  0 💤 ±0  0 ±0 
2 088 runs  ±0  2 088 ✔️ ±0  0 💤 ±0  0 ±0 

Results for commit a2c7ce2. ± Comparison against base commit a8386cc.

@gpachecoNI gpachecoNI merged commit a4423d2 into main Jun 30, 2023
@gpachecoNI gpachecoNI deleted the users/gpacheco/UpdateExamples branch June 30, 2023 17:35
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