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.

Sign up for GitHub

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

Jump to bottom

build: GitHub Copilot feature only in prerelease version #12885

Merged
merged 2 commits into from
Dec 10, 2024
Merged

build: GitHub Copilot feature only in prerelease version #12885

merged 2 commits into from
Dec 10, 2024

Conversation

yuqizhou77
Copy link
Contributor

@yuqizhou77 yuqizhou77 commented Dec 10, 2024 •
edited
Loading

  • cherrypick two commits from dev branch.

@yuqizhou77
fix: github copilot only shown if non stable release (#12843)
68a6af7 
* fix: github copilot only shown if non stable release

* fix: github copilot only shown if non stable release

* fix: github copilot only shown if non stable release

* fix: allow daily build
@yuqizhou77
ci: local commit (#12844)
b128602 
* fix: local commit

* fix: local commit

* fix: local commit
@codecov Codecov
Copy link

codecov bot commented Dec 10, 2024 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.04%. Comparing base (b44a1ea) to head (b128602).
Report is 3 commits behind head on release/5.12.

Additional details and impacted files

Impacted file tree graph

@@              Coverage Diff              @@
##           release/5.12   #12885   +/-   ##
=============================================
  Coverage         90.04%   90.04%           
=============================================
  Files               576      577    +1     
  Lines             34196    34198    +2     
  Branches           6841     6868   +27     
=============================================
+ Hits              30792    30794    +2     
  Misses             1501     1501           
  Partials           1903     1903
Files with missing lines Coverage Δ
...scode-extension/src/releaseBasedFeatureSettings.ts 100.00% <100.00%> (ø)

@yuqizhou77 yuqizhou77 marked this pull request as ready for review December 10, 2024 09:05
@MSFT-yiz MSFT-yiz merged commit c9080aa into release/5.12 Dec 10, 2024
25 checks passed
@MSFT-yiz MSFT-yiz deleted the yuqzho/stable-gce branch December 10, 2024 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wenytang-ms wenytang-ms wenytang-ms approved these changes

@MSFT-yiz MSFT-yiz MSFT-yiz approved these changes

@tecton tecton tecton approved these changes

@yiqing-zhao yiqing-zhao Awaiting requested review from yiqing-zhao yiqing-zhao is a code owner

@HuihuiWu-Microsoft HuihuiWu-Microsoft Awaiting requested review from HuihuiWu-Microsoft HuihuiWu-Microsoft is a code owner

@1openwindow 1openwindow Awaiting requested review from 1openwindow 1openwindow is a code owner

@nliu-ms nliu-ms Awaiting requested review from nliu-ms nliu-ms is a code owner

@LongOddCode LongOddCode Awaiting requested review from LongOddCode LongOddCode is a code owner

@Siglud Siglud Awaiting requested review from Siglud Siglud is a code owner

@qinezh qinezh Awaiting requested review from qinezh qinezh is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@yuqizhou77 @tecton @MSFT-yiz @wenytang-ms