Skip to content

[QUERY] [ENG] Should we have CI to warn that commit id in tsp-location.yaml is not on spec main branch? #36851

Open
@weidongxu-microsoft

Description

@weidongxu-microsoft

Query/Question

Should we have CI to warn that commit id in tsp-location.yaml is not on spec main branch?

E.g. in #36693
tsp-location.yaml https://github.com/Azure/azure-sdk-for-java/blob/8b6fca8f9b4486b61a92d709707e8ce54c900439/sdk/openai/azure-ai-openai/tsp-location.yaml

$ git branch -a --contains dd2d1e8957ac6654272137e8d5874eacafd80a5f
  remotes/upstream/user/travisw/aoai-whisper-mvp-spec

We probably wish to have a warning on this.

Usually we do not want to release SDK based on an unmerged PR.

Why is this not a Bug or a feature Request?
A clear explanation of why is this not a bug or a feature request?

Setup (please complete the following information if applicable):

  • OS: [e.g. iOS]
  • IDE: [e.g. IntelliJ]
  • Library/Libraries: [e.g. com.azure:azure-core:1.16.0 (groupId:artifactId:version)]

Information Checklist
Kindly make sure that you have added all the following information above and checkoff the required fields otherwise we will treat the issuer as an incomplete report

  • Query Added
  • Setup information Added

Metadata

Metadata

Assignees

No one assigned

    Labels

    EngSysThis issue is impacting the engineering system.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions