Skip to content

[AutoPR sdk-messaging/eventgrid/azsystemevents]-generated-from-SDK Generation - Go-6132552 - #26546

Closed
Azure SDK Bot (azure-sdk) wants to merge 5 commits into
mainfrom
sdkauto/sdk-messaging-eventgrid/azsystemevents-6132552
Closed

[AutoPR sdk-messaging/eventgrid/azsystemevents]-generated-from-SDK Generation - Go-6132552#26546
Azure SDK Bot (azure-sdk) wants to merge 5 commits into
mainfrom
sdkauto/sdk-messaging-eventgrid/azsystemevents-6132552

Conversation

@azure-sdk

Copy link
Copy Markdown
Collaborator

Configurations: 'specification/eventgrid/data-plane/EventGridSystemEvents/tspconfig.yaml', API Version: 2024-01-01, SDK Release Type: beta, and CommitSHA: '964e827a2beb3838001819532f1bafac405c520f' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=6132552 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.

…ents/tspconfig.yaml', API Version: 2024-01-01, SDK Release Type: beta, and CommitSHA: '964e827a2beb3838001819532f1bafac405c520f' in SpecRepo: 'https://github.com/Azure/azure-rest-api-specs' Pipeline run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=6132552 Refer to https://eng.ms/docs/products/azure-developer-experience/develop/sdk-release/sdk-release-prerequisites to prepare for SDK release.
@github-actions

github-actions Bot commented Apr 9, 2026

Copy link
Copy Markdown

API Change Check

APIView identified API level changes in this PR and created the following API reviews

sdk/messaging/eventgrid/azsystemevents

@jliusan

Copy link
Copy Markdown
Contributor

Copilot please fix CI failure

@jliusan

Copy link
Copy Markdown
Contributor

Copilot resolve the merge conflicts in this pull request

Co-authored-by: jliusan <6267860+jliusan@users.noreply.github.com>

Copilot AI commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Copilot please fix CI failure

Fixed in commit b36eac7. I restored sdk/messaging/eventgrid/azsystemevents/time_rfc3339.go, which was missing and causing compile failures (populateDateTimeRFC3339/unpopulateDateTimeRFC3339 undefined). I also verified with go test ./... in the module.

Copilot AI and others added 2 commits June 1, 2026 06:43
Co-authored-by: jliusan <6267860+jliusan@users.noreply.github.com>
Co-authored-by: jliusan <6267860+jliusan@users.noreply.github.com>

Copilot AI commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Copilot resolve the merge conflicts in this pull request

Resolved merge conflicts in commit 9706ca7. I accepted the main branch changes for:

  • models.go: Updated field naming (ToBsuid/FromBsuid per new emitter conventions)
  • models_serde.go: Updated serialization code using generic time helpers
  • testdata/_metadata.json: Emitter version 0.11.0
  • tsp-location.yaml: Spec commit 7089f3d2

Tests pass successfully.

@jliusan

Copy link
Copy Markdown
Contributor

Copilot fix ci failure

Co-authored-by: jliusan <6267860+jliusan@users.noreply.github.com>

Copilot AI commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Copilot fix ci failure

Fixed in commit 4e9b1e5. The CI analyze job was failing because the required doc.go file was missing from the azsystemevents module. I've added the file with appropriate package documentation.

@jliusan

Copy link
Copy Markdown
Contributor

Copilot revert to this commit:605c4a2a8b5867d3aac9e9b2d7267d8ac35de946

Copilot AI commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

Copilot revert to this commit:605c4a2a8b5867d3aac9e9b2d7267d8ac35de946

To revert to commit 605c4a2, I need to force-push the branch, which requires special permissions.

As an alternative, I can create a new commit that reverts all my previous changes (restoring time_rfc3339.go removal, undoing merge conflict resolution, and removing doc.go). This would achieve the same end state as commit 605c4a2 without requiring a force push.

Would you like me to proceed with the revert commit approach, or would you prefer to handle the force-push to 605c4a2 manually?

@github-actions

github-actions Bot commented Aug 7, 2026

Copy link
Copy Markdown

Hi Azure SDK Bot (@azure-sdk). Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days.

@github-actions github-actions Bot added the no-recent-activity There has been no recent activity on this issue. label Aug 7, 2026
@github-actions

Copy link
Copy Markdown

Hi Azure SDK Bot (@azure-sdk). Thank you for your contribution. Since there hasn't been recent engagement, we're going to close this out. Feel free to respond with a comment containing /reopen if you'd like to continue working on these changes. Please be sure to use the command to reopen or remove the no-recent-activity label; otherwise, this is likely to be closed again with the next cleanup pass.

@github-actions github-actions Bot closed this Aug 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Event Grid no-recent-activity There has been no recent activity on this issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants