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

Add a job to build Bicep CLI for linux-musl-x64 #1752

Merged
merged 13 commits into from
Mar 10, 2021

Conversation

shenglol
Copy link
Contributor

@shenglol shenglol commented Mar 6, 2021

Contributing a Pull Request

Contributing a feature

  • I have opened a new issue for the proposal, or commented on an existing one, and ensured that the Bicep maintainers are good with the design of the feature being implemented
  • I have included "Fixes #{issue_number}" in the PR description, so GitHub can link to the issue and close it when the PR is merged
  • I have appropriate test coverage of my new feature

This is the first step to address #1683.

.github/workflows/build.yml Outdated Show resolved Hide resolved
@miqm
Copy link
Collaborator

miqm commented Mar 8, 2021

FYI you have typo in PR title ;)

@anthony-c-martin anthony-c-martin changed the title Add a job to build Bicpe CLI for linux-musl-x64 Add a job to build Bicep CLI for linux-musl-x64 Mar 8, 2021
@codecov-io
Copy link

codecov-io commented Mar 10, 2021

Codecov Report

Merging #1752 (3974295) into main (25339dd) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1752   +/-   ##
=======================================
  Coverage   95.03%   95.03%           
=======================================
  Files         371      371           
  Lines       21692    21692           
  Branches       15       15           
=======================================
  Hits        20615    20615           
  Misses       1077     1077           
Flag Coverage Δ
dotnet 95.47% <ø> (ø)
typescript 26.61% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

@shenglol shenglol requested a review from majastrz March 10, 2021 19:04
@majastrz majastrz merged commit f87fd5f into main Mar 10, 2021
@majastrz majastrz deleted the shenglol/build-linux-musl-x64 branch March 10, 2021 21:17
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