Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

test: Wrap provisionerd channel closes in sync.Once#1181

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
kylecarbs merged 1 commit intomainfromwrapclose
Apr 26, 2022

Conversation

kylecarbs
Copy link
Member

This caused a few flakes, so figured I'd tackle all of them:
https://github.com/coder/coder/runs/6167856950?check_suite_focus=true#step:9:246

@kylecarbskylecarbs requested a review fromcoadlerApril 26, 2022 02:12
@kylecarbskylecarbs self-assigned thisApr 26, 2022
@kylecarbskylecarbsenabled auto-merge (squash)April 26, 2022 02:12
@codecov
Copy link

codecovbot commentedApr 26, 2022
edited
Loading

Codecov Report

Merging#1181 (abbcea5) intomain (744a00a) willincrease coverage by0.18%.
The diff coverage isn/a.

@@            Coverage Diff             @@##             main    #1181      +/-   ##==========================================+ Coverage   66.16%   66.34%   +0.18%==========================================  Files         257      263       +6       Lines       16394    16615     +221       Branches      156      156              ==========================================+ Hits        10847    11024     +177- Misses       4417     4453      +36- Partials     1130     1138       +8
FlagCoverage Δ
unittest-go-macos-latest53.60% <ø> (?)
unittest-go-postgres-65.49% <ø> (-0.02%)⬇️
unittest-go-ubuntu-latest56.23% <ø> (ø)
unittest-go-windows-202253.27% <ø> (?)
unittest-js67.45% <ø> (ø)
Impacted FilesCoverage Δ
peerbroker/dial.go77.04% <0.00%> (-6.56%)⬇️
peerbroker/proxy.go58.72% <0.00%> (-3.49%)⬇️
coderd/workspaceagents.go55.59% <0.00%> (-2.56%)⬇️
provisioner/terraform/provision.go70.96% <0.00%> (-0.44%)⬇️
agent/usershell/usershell_darwin.go100.00% <0.00%> (ø)
agent/usershell/usershell_windows.go83.33% <0.00%> (ø)
cli/cliui/cliui_darwin.go0.00% <0.00%> (ø)
cli/ssh_windows.go0.00% <0.00%> (ø)
pty/pty_windows.go70.14% <0.00%> (ø)
pty/start_windows.go66.95% <0.00%> (ø)
... and12 more

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last update744a00a...abbcea5. Read thecomment docs.

@kylecarbskylecarbs requested a review fromEmyrkApril 26, 2022 14:43
@kylecarbskylecarbs merged commit603b7da intomainApr 26, 2022
@kylecarbskylecarbs deleted the wrapclose branchApril 26, 2022 14:44
kylecarbs added a commit that referenced this pull requestJun 10, 2022
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@EmyrkEmyrkEmyrk approved these changes

@coadlercoadlerAwaiting requested review from coadler

Assignees

@kylecarbskylecarbs

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants
@kylecarbs@Emyrk

[8]ページ先頭

©2009-2025 Movatter.jp