Skip to content

Commit 1991388

Browse files
committed
Add new article code reviews
Also: * adjusted css slightly * added new docker container
1 parent 72447da commit 1991388

File tree

8 files changed

+85
-48
lines changed

8 files changed

+85
-48
lines changed

Gemfile

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
source 'https://rubygems.org'
22

3-
gem "github-pages", '193', group: :jekyll_plugins
3+
gem 'github-pages', group: :jekyll_plugins
44

55
# enable tzinfo-data for local build
66
# gem 'tzinfo-data', platforms: [:mingw, :mswin, :x64_mingw]

Gemfile.lock

Lines changed: 37 additions & 43 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
GEM
22
remote: https://rubygems.org/
33
specs:
4-
activesupport (4.2.10)
4+
activesupport (4.2.11.1)
55
i18n (~> 0.7)
66
minitest (~> 5.1)
77
thread_safe (~> 0.3, >= 0.3.4)
88
tzinfo (~> 1.1)
9-
addressable (2.6.0)
10-
public_suffix (>= 2.0.2, < 4.0)
9+
addressable (2.7.0)
10+
public_suffix (>= 2.0.2, < 5.0)
1111
coffee-script (2.4.1)
1212
coffee-script-source
1313
execjs
@@ -16,40 +16,38 @@ GEM
1616
commonmarker (0.17.13)
1717
ruby-enum (~> 0.5)
1818
concurrent-ruby (1.1.5)
19-
dnsruby (1.61.2)
19+
dnsruby (1.61.3)
2020
addressable (~> 2.5)
2121
em-websocket (0.5.1)
2222
eventmachine (>= 0.12.9)
2323
http_parser.rb (~> 0.6.0)
2424
ethon (0.12.0)
2525
ffi (>= 1.3.0)
2626
eventmachine (1.2.7)
27-
eventmachine (1.2.7-x64-mingw32)
2827
execjs (2.7.0)
29-
faraday (0.15.4)
28+
faraday (0.17.0)
3029
multipart-post (>= 1.2, < 3)
3130
ffi (1.11.1)
32-
ffi (1.11.1-x64-mingw32)
3331
forwardable-extended (2.6.0)
3432
gemoji (3.0.1)
35-
github-pages (193)
36-
activesupport (= 4.2.10)
37-
github-pages-health-check (= 1.8.1)
38-
jekyll (= 3.7.4)
33+
github-pages (202)
34+
activesupport (= 4.2.11.1)
35+
github-pages-health-check (= 1.16.1)
36+
jekyll (= 3.8.5)
3937
jekyll-avatar (= 0.6.0)
4038
jekyll-coffeescript (= 1.1.1)
41-
jekyll-commonmark-ghpages (= 0.1.5)
39+
jekyll-commonmark-ghpages (= 0.1.6)
4240
jekyll-default-layout (= 0.1.4)
4341
jekyll-feed (= 0.11.0)
4442
jekyll-gist (= 1.5.0)
45-
jekyll-github-metadata (= 2.9.4)
43+
jekyll-github-metadata (= 2.12.1)
4644
jekyll-mentions (= 1.4.1)
4745
jekyll-optional-front-matter (= 0.3.0)
4846
jekyll-paginate (= 1.1.0)
4947
jekyll-readme-index (= 0.2.0)
5048
jekyll-redirect-from (= 0.14.0)
51-
jekyll-relative-links (= 0.5.3)
52-
jekyll-remote-theme (= 0.3.1)
49+
jekyll-relative-links (= 0.6.0)
50+
jekyll-remote-theme (= 0.4.0)
5351
jekyll-sass-converter (= 1.5.2)
5452
jekyll-seo-tag (= 2.5.0)
5553
jekyll-sitemap (= 1.2.0)
@@ -68,28 +66,28 @@ GEM
6866
jekyll-theme-tactile (= 0.1.1)
6967
jekyll-theme-time-machine (= 0.1.1)
7068
jekyll-titles-from-headings (= 0.5.1)
71-
jemoji (= 0.10.1)
69+
jemoji (= 0.10.2)
7270
kramdown (= 1.17.0)
7371
liquid (= 4.0.0)
7472
listen (= 3.1.5)
7573
mercenary (~> 0.3)
7674
minima (= 2.5.0)
77-
nokogiri (>= 1.8.2, < 2.0)
78-
rouge (= 2.2.1)
75+
nokogiri (>= 1.10.4, < 2.0)
76+
rouge (= 3.11.0)
7977
terminal-table (~> 1.4)
80-
github-pages-health-check (1.8.1)
78+
github-pages-health-check (1.16.1)
8179
addressable (~> 2.3)
8280
dnsruby (~> 1.60)
8381
octokit (~> 4.0)
84-
public_suffix (~> 2.0)
82+
public_suffix (~> 3.0)
8583
typhoeus (~> 1.3)
86-
html-pipeline (2.11.0)
84+
html-pipeline (2.12.0)
8785
activesupport (>= 2)
8886
nokogiri (>= 1.4)
8987
http_parser.rb (0.6.0)
9088
i18n (0.9.5)
9189
concurrent-ruby (~> 1.0)
92-
jekyll (3.7.4)
90+
jekyll (3.8.5)
9391
addressable (~> 2.4)
9492
colorator (~> 1.0)
9593
em-websocket (~> 0.5)
@@ -110,18 +108,18 @@ GEM
110108
jekyll-commonmark (1.3.1)
111109
commonmarker (~> 0.14)
112110
jekyll (>= 3.7, < 5.0)
113-
jekyll-commonmark-ghpages (0.1.5)
111+
jekyll-commonmark-ghpages (0.1.6)
114112
commonmarker (~> 0.17.6)
115-
jekyll-commonmark (~> 1)
116-
rouge (~> 2)
113+
jekyll-commonmark (~> 1.2)
114+
rouge (>= 2.0, < 4.0)
117115
jekyll-default-layout (0.1.4)
118116
jekyll (~> 3.0)
119117
jekyll-feed (0.11.0)
120118
jekyll (~> 3.3)
121119
jekyll-gist (1.5.0)
122120
octokit (~> 4.2)
123-
jekyll-github-metadata (2.9.4)
124-
jekyll (~> 3.1)
121+
jekyll-github-metadata (2.12.1)
122+
jekyll (~> 3.4)
125123
octokit (~> 4.0, != 4.4.0)
126124
jekyll-mentions (1.4.1)
127125
html-pipeline (~> 2.3)
@@ -133,9 +131,10 @@ GEM
133131
jekyll (~> 3.0)
134132
jekyll-redirect-from (0.14.0)
135133
jekyll (~> 3.3)
136-
jekyll-relative-links (0.5.3)
134+
jekyll-relative-links (0.6.0)
137135
jekyll (~> 3.3)
138-
jekyll-remote-theme (0.3.1)
136+
jekyll-remote-theme (0.4.0)
137+
addressable (~> 2.0)
139138
jekyll (~> 3.5)
140139
rubyzip (>= 1.2.1, < 3.0)
141140
jekyll-sass-converter (1.5.2)
@@ -189,7 +188,7 @@ GEM
189188
jekyll (~> 3.3)
190189
jekyll-watch (2.2.1)
191190
listen (~> 3.0)
192-
jemoji (0.10.1)
191+
jemoji (0.10.2)
193192
gemoji (~> 3.0)
194193
html-pipeline (~> 2.2)
195194
jekyll (~> 3.0)
@@ -200,28 +199,24 @@ GEM
200199
rb-inotify (~> 0.9, >= 0.9.7)
201200
ruby_dep (~> 1.2)
202201
mercenary (0.3.6)
203-
mini_portile2 (2.5.1)
202+
mini_portile2 (2.4.0)
204203
minima (2.5.0)
205204
jekyll (~> 3.5)
206205
jekyll-feed (~> 0.9)
207206
jekyll-seo-tag (~> 2.1)
208-
minitest (5.11.3)
207+
minitest (5.13.0)
209208
multipart-post (2.1.1)
210-
nokogiri (1.11.5)
211-
mini_portile2 (~> 2.5.0)
212-
racc (~> 1.4)
213-
nokogiri (1.11.5-x64-mingw32)
214-
racc (~> 1.4)
209+
nokogiri (1.10.5)
210+
mini_portile2 (~> 2.4.0)
215211
octokit (4.14.0)
216212
sawyer (~> 0.8.0, >= 0.5.3)
217213
pathutil (0.16.2)
218214
forwardable-extended (~> 2.6)
219-
public_suffix (2.0.5)
220-
racc (1.5.2)
215+
public_suffix (3.1.1)
221216
rb-fsevent (0.10.3)
222217
rb-inotify (0.10.0)
223218
ffi (~> 1.0)
224-
rouge (2.2.1)
219+
rouge (3.11.0)
225220
ruby-enum (0.7.2)
226221
i18n
227222
ruby_dep (1.5.0)
@@ -246,12 +241,11 @@ GEM
246241

247242
PLATFORMS
248243
ruby
249-
x64-mingw32
250244

251245
DEPENDENCIES
252-
github-pages (= 193)
246+
github-pages
253247
jekyll-paginate
254248
jekyll-redirect-from
255249

256250
BUNDLED WITH
257-
2.2.19
251+
1.16.3

_includes/author.html

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,9 +6,10 @@ <h4>About the Author:</h4>
66
<div class="authordescription">
77
<p>
88
Marcus is fascinated by technology. He loves learning new things.<br>
9-
He is a Software Engineer at <a href="https://www.daimler-tss.com/en/" rel="nofollow">Daimler TSS</a> and has a personal blog at <a href="https://marcuseisele.com/" rel="nofollow">MarcusEisele.com</a>.
9+
He is a Software Engineer at <a href="https://www.bosch.com/" rel="nofollow">Bosch</a> and has a
10+
personal blog at <a href="https://marcuseisele.com/" rel="nofollow">MarcusEisele.com</a>.
1011
</p>
11-
12+
1213
</div>
1314
</div>
1415
</div>
Lines changed: 38 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,38 @@
1+
---
2+
layout: post
3+
title: Tips for Improving Code Review Efficiency
4+
share-img: /img/content/code-review-share.jpg
5+
image: /img/content/code-review-share.jpg
6+
bigimg: /img/content/code-review.jpg
7+
permalink: /tips-for-improving-code-review-efficiency/
8+
tags: [development, code review]
9+
---
10+
11+
Code reviews are a vital part of the software development process. They ensure that the code is correct, efficient, and maintainable. However, code reviews can also be time-consuming and often slow down the development process. In this article, we will share some tips for improving code review efficiency.
12+
13+
## 1. Set Clear Guidelines and Standards
14+
It's essential to establish clear guidelines and standards for code reviews within your team. These guidelines should cover:
15+
16+
* What to look for in detail
17+
* The level of detail required (e.g. testing the feature vs just looking at the code)
18+
* How to give constructive feedback
19+
20+
By establishing these standards, you will reduce the amount of time spent on back-and-forth discussions during code reviews.
21+
22+
## 2. Make Small, Regular Changes
23+
Rather than making big changes to the codebase all at once, it's better to make small, regular changes. Small changes tend to be reviewed faster and more easily, reducing the time spent on code reviews. Furthermore, smaller changes are easier to understand, which makes it easier for reviewers to provide constructive feedback.
24+
25+
## 3. Use Static Code Analysis Tools
26+
Static Code analysis tools can help identify potential issues and provide suggestions for improvement. By using these tools, you can catch errors and improve the quality of your code before it's even reviewed by your team.
27+
28+
## 4. Prioritize Code Reviews
29+
Make code reviews a priority within your team. Set aside time for reviewers to focus solely on code reviews, and make sure they have the necessary tools and resources to do so effectively. This will help ensure that code reviews are completed in a timely manner and don't slow down the development process.
30+
31+
## 5. Foster a Collaborative Environment
32+
Code reviews should be viewed as a collaborative process rather than a way to find faults or assign blame. Encourage your team to ask questions and provide constructive feedback during code reviews. By creating a collaborative environment, you can improve the quality of your code and foster a culture of continuous improvement.
33+
34+
# Conclusion
35+
36+
Code reviews are an essential part of the software development process. By following these tips, you can improve code review efficiency and reduce the amount of time spent on back-and-forth discussions. Remember to set clear guidelines and standards, make small, regular changes, use code analysis tools, prioritize code reviews, and foster a collaborative environment.
37+
38+
We hope you found these tips helpful. Do you have any other tips for improving code review efficiency? Let us know in the comments below!

css/custom.css

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,12 @@ pre > code {
4343
}
4444

4545
.authorbox {
46-
background-color: #F5F5F5 !important;
46+
background-color: #FFF !important;
47+
border-color: #FFF;
4748
border-radius: 10px;
49+
border-width: 1px;
50+
padding: 10px 10px 10px 15px;
51+
filter: drop-shadow(0.5rem 0.5rem 1.5rem rgba(0, 0, 0, 0.2));
4852
}
4953

5054
.authorname {

img/content/code-review-share.jpg

77 KB
Loading

img/content/code-review.jpg

61.7 KB
Loading

runLocally.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
#!/usr/bin/env bash
22

3-
bundle exec jekyll serve --incremental
3+
docker run -it --rm -v "$PWD":/usr/src/app --publish 127.0.0.1:3000:4000 starefossen/github-pages

0 commit comments

Comments
 (0)