Skip to content
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

Replace make as a script running with just #903

Merged
merged 9 commits into from May 10, 2022
Merged

Replace make as a script running with just #903

merged 9 commits into from May 10, 2022

Conversation

clux
Copy link
Member

@clux clux commented May 10, 2022

To follow conventions used in rest of org.

..triggered by needing to make a new target and stopped writing make pretty much everywhere else.

clux added 2 commits May 10, 2022 10:14
To follow conventions used in rest of org.

Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
@clux clux added the changelog-exclude changelog excluded prs label May 10, 2022
@clux clux added this to the 0.72.0 milestone May 10, 2022
@codecov-commenter
Copy link

Codecov Report

Merging #903 (612f37f) into master (c976f9e) will increase coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #903      +/-   ##
==========================================
+ Coverage   70.35%   70.37%   +0.02%     
==========================================
  Files          62       62              
  Lines        4291     4291              
==========================================
+ Hits         3019     3020       +1     
+ Misses       1272     1271       -1     
Impacted Files Coverage Δ
kube-runtime/src/wait.rs 70.00% <0.00%> (+2.00%) ⬆️

@clux clux requested a review from kazk May 10, 2022 09:53
Signed-off-by: clux <sszynrae@gmail.com>
@olix0r
Copy link
Contributor

olix0r commented May 10, 2022

Perhaps add just to the devcontainer? It may also be helpful to include a link/installation instructions somewhere (since it's effectively unsearchable).

Signed-off-by: clux <sszynrae@gmail.com>
@clux
Copy link
Member Author

clux commented May 10, 2022

Have added it to the devcontainer. The third party tools are currently documented on https://kube.rs/tools/ which sits under the contributing section. Do you think that's OK?

clux added 2 commits May 10, 2022 20:06
Signed-off-by: clux <sszynrae@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
justfile Show resolved Hide resolved
.devcontainer/Dockerfile Outdated Show resolved Hide resolved
clux and others added 3 commits May 10, 2022 21:18
Co-authored-by: kazk <kazk.dev@gmail.com>
Co-authored-by: kazk <kazk.dev@gmail.com>
Signed-off-by: clux <sszynrae@gmail.com>
@clux clux merged commit dbac80d into master May 10, 2022
@clux clux deleted the s-make-just branch May 10, 2022 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog-exclude changelog excluded prs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants