Skip to content

Only show automodel button for Java#2656

Merged
charisk merged 1 commit intomainfrom
charisk/show-llm-button
Jul 31, 2023
Merged

Only show automodel button for Java#2656
charisk merged 1 commit intomainfrom
charisk/show-llm-button

Conversation

@charisk
Copy link
Copy Markdown
Contributor

@charisk charisk commented Jul 31, 2023

We currently show the button for both Java and C# and the solution doesn't work for C#.

Checklist

N/A:

  • CHANGELOG.md has been updated to incorporate all user visible changes made by this pull request.
  • Issues have been created for any UI or other user-facing changes made by this pull request.
  • [Maintainers only] If this pull request makes user-facing changes that require documentation changes, open a corresponding docs pull request in the github/codeql repo and add the ready-for-doc-review label there.

@charisk charisk requested a review from starcke July 31, 2023 07:40
@charisk charisk requested a review from a team as a code owner July 31, 2023 07:40
starcke
starcke approved these changes Jul 31, 2023
View reviewed changes
Copy link
Copy Markdown
Contributor

@starcke starcke left a comment

Choose a reason for hiding this comment

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

👍

@charisk charisk enabled auto-merge (squash) July 31, 2023 07:46
@charisk charisk merged commit 5427b57 into main Jul 31, 2023
@charisk charisk deleted the charisk/show-llm-button branch July 31, 2023 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@starcke starcke starcke approved these changes

@robertbrignull robertbrignull robertbrignull approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@charisk @starcke @robertbrignull