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

chore: update semantic types#1030

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
greyscaled merged 5 commits intomainfromvapurrmaid/semantic-types
Apr 15, 2022
Merged

Conversation

greyscaled
Copy link
Contributor

@greyscaledgreyscaled commentedApr 15, 2022
edited
Loading

Summary:

PRs like#1025 feel like they deserve anexample: type, but we didn't have one.
Furthermore our definitions for correct, fix, and hotfix were stale.

Summary:PRs like#1025 feel like they deserve a doc: type, but we didn't have one.Furthermore our definitions for correct and fix were stale.
@greyscaledgreyscaled self-assigned thisApr 15, 2022
@codecov
Copy link

codecovbot commentedApr 15, 2022
edited
Loading

Codecov Report

Merging#1030 (a342411) intomain (88e30be) willincrease coverage by9.66%.
The diff coverage isn/a.

@@            Coverage Diff             @@##             main    #1030      +/-   ##==========================================+ Coverage   57.70%   67.37%   +9.66%==========================================  Files         259      259                Lines       15059    15059                Branches      151      151              ==========================================+ Hits         8690    10146    +1456+ Misses       5426     3893    -1533- Partials      943     1020      +77
FlagCoverage Δ
unittest-go-macos-latest53.56% <ø> (+0.09%)⬆️
unittest-go-postgres-66.62% <ø> (?)
unittest-go-ubuntu-latest56.29% <ø> (-0.02%)⬇️
unittest-go-windows-202253.03% <ø> (+0.04%)⬆️
unittest-js68.20% <ø> (ø)
Impacted FilesCoverage Δ
peerbroker/listen.go84.16% <0.00%> (-3.34%)⬇️
peerbroker/proxy.go57.55% <0.00%> (-0.59%)⬇️
peer/conn.go79.69% <0.00%> (-0.26%)⬇️
peer/channel.go83.81% <0.00%> (+0.57%)⬆️
coderd/workspaceagents.go59.82% <0.00%> (+1.33%)⬆️
coderd/templates.go62.29% <0.00%> (+1.63%)⬆️
cli/cliui/provisionerjob.go78.57% <0.00%> (+2.14%)⬆️
provisioner/echo/serve.go56.80% <0.00%> (+2.39%)⬆️
provisionerd/provisionerd.go80.73% <0.00%> (+2.64%)⬆️
coderd/provisionerdaemons.go63.46% <0.00%> (+4.11%)⬆️
... and9 more

Continue to review full report at Codecov.

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

Copy link
Member

@kylecarbskylecarbs left a comment

Choose a reason for hiding this comment

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

I'd prefer we make anexample tag for now. It'd be cool to see the delta of example changes between releases anyways!

@greyscaled
Copy link
ContributorAuthor

I'd prefer we make anexample tag for now. It'd be cool to see the delta of example changes between releases anyways!

sounds great to me!

Copy link
Member

@kylecarbskylecarbs left a comment

Choose a reason for hiding this comment

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

LGTM!

@greyscaledgreyscaled merged commit148e7cd intomainApr 15, 2022
@greyscaledgreyscaled deleted the vapurrmaid/semantic-types branchApril 15, 2022 20:26
@missknissmisskniss added this to theV2 Beta milestoneMay 15, 2022
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@presleyppresleyppresleyp approved these changes

@johnstcnjohnstcnjohnstcn approved these changes

@kylecarbskylecarbskylecarbs approved these changes

Assignees

@greyscaledgreyscaled

Labels
None yet
Projects
None yet
Milestone
V2 Beta
Development

Successfully merging this pull request may close these issues.

5 participants
@greyscaled@presleyp@johnstcn@kylecarbs@misskniss

[8]ページ先頭

©2009-2025 Movatter.jp