<div id="readability-page-1" class="page"><div>
			
<main role="main">
	<article>
		<div>
			<p><code>Vercel</code>은 각자 사는 나라의 연휴에 맞춰서 노는 시스템이다.
그래서 오늘은 노는 날이다.</p>
<hr>
<p>Shopify의 PR을 리뷰해줬다.</p>
<p><a href="https://github.com/Shopify/swcify/pull/27#pullrequestreview-769817506">https://github.com/Shopify/swcify/pull/27#pullrequestreview-769817506</a></p>
<p>리뷰할 게 많아서 시간이 좀 걸렸다.
1시간 반 정도?</p>
<p>우선 <code>Deref</code>의 사용법을 모르고 있는 것 같아서 설명해줬다.</p>
<p><img src="https://kdy1.github.io/post/2021/10/shopify/1-review/ss-deref.png" alt="Screeenshot"></p>
<hr>
<p>러스트 플레이그라운드를 이용해서 <code>filter_map</code>을 어떤 식으로 써야하는지 <code>?</code> 연산자를 쓴 이유가 뭔지도 설명해줬다.</p>
<p><img src="https://kdy1.github.io/post/2021/10/shopify/1-review/ss-filter-map.png" alt="Screeenshot"></p>
<p>샤진엔 안 나왔는데, 기존 코드는 <code>map</code>을 한 다음에 아래에서 <code>retain</code>을 이용해서 데이터를 지운다.</p>
<hr>
<p>그리고 가독성을 개선하는 방법에 대해서도 많이 설명해줬다.</p>
<p>가독성까지는 오바일 수도 있지만 그래도 러스트 전문가로 컨설턴트 계약을 한 것이니까 사소한 가독성 이슈도 잡아줬다.
리뷰하고 나서 보니까 58개였다.</p>
<p>코드 리뷰했더니 피곤해서 잠깐 쉴 생각이다.</p>

		</div>
	</article>
</main>






			</div></div>

Math && Programming

Vercel은 각자 사는 나라의 연휴에 맞춰서 노는 시스템이다. 그래서 오늘은 노는 날이다. Shopify의 PR을 리뷰해줬다. https://github.com/Shopify/swcify/pull/27#pullrequestreview-769817506 리뷰할 게 많아서 시간이 좀 걸

Shopify 첫번재 작업: PR 리뷰

댓글