Skip to content

chore: Rename Projects to Templates #880

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 2 commits into from
Apr 6, 2022
Merged

chore: Rename Projects to Templates #880

merged 2 commits into from
Apr 6, 2022

Conversation

kylecarbs
Copy link
Member

Customer feedback indicated "projects" is a confusing name.
After querying the team internally, it seemed unanimous
that it is indeed a confusing name.

Here's to a lil less confusion @ashmeer7 🥂

@kylecarbs kylecarbs requested review from johnstcn and f0ssel April 5, 2022 21:35
@kylecarbs kylecarbs self-assigned this Apr 5, 2022
@codecov
Copy link

codecov bot commented Apr 5, 2022

Codecov Report

Merging #880 (ed85167) into main (584c8b4) will increase coverage by 0.22%.
The diff coverage is 73.89%.

@@            Coverage Diff             @@
##             main     #880      +/-   ##
==========================================
+ Coverage   65.52%   65.75%   +0.22%     
==========================================
  Files         216      216              
  Lines       13733    13733              
  Branches      103      103              
==========================================
+ Hits         8999     9030      +31     
+ Misses       3807     3780      -27     
+ Partials      927      923       -4     
Flag Coverage Δ
unittest-go- 65.63% <75.55%> (+0.52%) ⬆️
unittest-go-macos-latest 52.76% <52.26%> (-0.03%) ⬇️
unittest-go-ubuntu-latest 55.22% <52.26%> (-0.05%) ⬇️
unittest-go-windows-2022 52.13% <52.26%> (+0.03%) ⬆️
unittest-js 58.83% <14.28%> (ø)
Impacted Files Coverage Δ
cli/cliui/parameter.go 0.00% <0.00%> (ø)
cli/parameters.go 16.07% <0.00%> (ø)
cli/workspacelist.go 9.52% <0.00%> (ø)
cli/workspaceupdate.go 7.89% <0.00%> (ø)
coderd/httpmw/workspaceresourceparam.go 59.18% <ø> (ø)
coderd/parameters.go 63.04% <0.00%> (ø)
coderd/userpassword/userpassword.go 63.41% <ø> (ø)
codersdk/parameters.go 66.66% <ø> (ø)
codersdk/users.go 65.38% <ø> (ø)
codersdk/workspacebuilds.go 58.33% <ø> (ø)
... and 54 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 584c8b4...ed85167. Read the comment docs.

@kylecarbs
Copy link
Member Author

I somehow net positive one line. Sucks to suck I guess.

@kylecarbs kylecarbs requested a review from a team as a code owner April 5, 2022 21:49
Customer feedback indicated projects was a confusing name.
After querying the team internally, it seemed unanimous
that it is indeed a confusing name.

Here's for a lil less confusion @ashmeer7 🥂
Copy link
Contributor

@f0ssel f0ssel left a comment

Choose a reason for hiding this comment

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

yolo

@@ -23,7 +23,7 @@ const (

// Compare checks the equality of passwords from a hashed pbkdf2 string.
// This uses pbkdf2 to ensure FIPS 140-2 compliance. See:
// https://csrc.nist.gov/csrc/media/projects/cryptographic-module-validation-program/documents/security-policies/140sp2261.pdf
// https://csrc.nist.gov/csrc/media/templates/cryptographic-module-validation-program/documents/security-policies/140sp2261.pdf
Copy link
Member

@johnstcn johnstcn Apr 6, 2022

Choose a reason for hiding this comment

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

minor: 404

Copy link
Member Author

Choose a reason for hiding this comment

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

Impressive catch!

Copy link
Member

@johnstcn johnstcn left a comment

Choose a reason for hiding this comment

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

LGTM, just one broken URL in a comment (easy fix)

@ashmeer7
Copy link

ashmeer7 commented Apr 6, 2022

This was the fastest I've ever complained about something to it being in code 👍 👍

@kylecarbs kylecarbs merged commit 02ad3f1 into main Apr 6, 2022
@kylecarbs kylecarbs deleted the templates branch April 6, 2022 17:42
BrunoQuaresma pushed a commit that referenced this pull request Apr 6, 2022
Customer feedback indicated projects was a confusing name.
After querying the team internally, it seemed unanimous
that it is indeed a confusing name.

Here's for a lil less confusion @ashmeer7 🥂
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.

6 participants