Skip to content

chore: update semantic types - #1030

Merged
greyscaled merged 5 commits into
mainfrom
vapurrmaid/semantic-types
Apr 15, 2022
Merged

greyscaled merged 5 commits into
mainfrom
vapurrmaid/semantic-types

Conversation

@greyscaled

@greyscaled greyscaled commented Apr 15, 2022 •

Copy link
Copy Markdown
Contributor

Summary:

PRs like #1025 feel like they deserve an example: 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.
Comment thread .github/semantic.yaml Resolved
@greyscaled greyscaled self-assigned this Apr 15, 2022
@codecov

codecov Bot commented Apr 15, 2022 •

Copy link
Copy Markdown

Codecov Report

Merging #1030 (a342411) into main (88e30be) will increase coverage by 9.66%.
The diff coverage is n/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     
Flag Coverage Δ
unittest-go-macos-latest 53.56% <ø> (+0.09%) ⬆️
unittest-go-postgres- 66.62% <ø> (?)
unittest-go-ubuntu-latest 56.29% <ø> (-0.02%) ⬇️
unittest-go-windows-2022 53.03% <ø> (+0.04%) ⬆️
unittest-js 68.20% <ø> (ø)
Impacted Files Coverage Δ
peerbroker/listen.go 84.16% <0.00%> (-3.34%) ⬇️
peerbroker/proxy.go 57.55% <0.00%> (-0.59%) ⬇️
peer/conn.go 79.69% <0.00%> (-0.26%) ⬇️
peer/channel.go 83.81% <0.00%> (+0.57%) ⬆️
coderd/workspaceagents.go 59.82% <0.00%> (+1.33%) ⬆️
coderd/templates.go 62.29% <0.00%> (+1.63%) ⬆️
cli/cliui/provisionerjob.go 78.57% <0.00%> (+2.14%) ⬆️
provisioner/echo/serve.go 56.80% <0.00%> (+2.39%) ⬆️
provisionerd/provisionerd.go 80.73% <0.00%> (+2.64%) ⬆️
coderd/provisionerdaemons.go 63.46% <0.00%> (+4.11%) ⬆️
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 88e30be...a342411. Read the comment docs.

@kylecarbs kylecarbs left a comment

Copy link
Copy Markdown
Member

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 an example tag for now. It'd be cool to see the delta of example changes between releases anyways!

@greyscaled

Copy link
Copy Markdown
Contributor Author

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

sounds great to me!

Comment thread .github/semantic.yaml Resolved
Comment thread .github/semantic.yaml Resolved
@greyscaled
greyscaled requested review from a team, Emyrk, coadler, deansheather, f0ssel, johnstcn and kylecarbs and removed request for Emyrk, coadler, deansheather, f0ssel and johnstcn April 15, 2022 18:48

@kylecarbs kylecarbs left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM!

Comment thread .github/semantic.yaml Resolved
Comment thread .github/semantic.yaml Resolved
@greyscaled
greyscaled merged commit 148e7cd into main Apr 15, 2022
@greyscaled
greyscaled deleted the vapurrmaid/semantic-types branch April 15, 2022 20:26
@misskniss misskniss added this to the V2 Beta milestone May 15, 2022
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.

5 participants