Skip to content

Update main.go #86

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

Merged
merged 1 commit into from
Oct 28, 2024
Merged

Update main.go #86

merged 1 commit into from
Oct 28, 2024

Conversation

jason-shen
Copy link
Member

added comment

If you are serializing a candidate make sure to use ToJSON
Using Marshal will result in errors around sdpMid

Description

Reference issue

Fixes #...

@jason-shen jason-shen requested a review from Sean-Der February 4, 2021 11:15
@stv0g
Copy link
Member

stv0g commented May 11, 2023

Hi @jason-shen,

I see the CI tests failing. Do you mind to fix them and rebase your changes on the latest master?

If you are serializing a candidate make sure to use ToJSON
Using Marshal will result in errors around `sdpMid`
Copy link

codecov bot commented Oct 28, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (b4164e3) to head (f64faa5).
Report is 1 commits behind head on master.

Additional details and impacted files
@@          Coverage Diff           @@
##           master     #86   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files          25      25           
  Lines        2803    2803           
======================================
  Misses       2803    2803           
Flag Coverage Δ
go 0.00% <ø> (ø)
wasm 0.00% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Sean-Der Sean-Der merged commit f3b8c43 into master Oct 28, 2024
14 of 15 checks passed
@Sean-Der Sean-Der deleted the jason-shen-patch-1 branch October 28, 2024 03:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants