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

hyp3rflow님 기여 #5

Open
2 of 3 tasks
sanxiyn opened this issue Sep 25, 2022 · 7 comments
Open
2 of 3 tasks

hyp3rflow님 기여 #5

sanxiyn opened this issue Sep 25, 2022 · 7 comments
Assignees

Comments

@sanxiyn
Copy link
Member

sanxiyn commented Sep 25, 2022

  • 프로그래밍 언어 선정: Rust
  • 프로젝트 선정: Deno
  • 이슈 선정

JavaScript보다는 Rust에 기여하고 싶다는 의견 주셨습니다.

@hyp3rflow
Copy link

hyp3rflow commented Oct 2, 2022

deno/issues/13797
deno/issues/15176
deno/issues/15661
이런쪽으로 생각중인데, 막상 찾아보니까 다 어려워보이네요 :(

@sanxiyn
Copy link
Member Author

sanxiyn commented Oct 2, 2022

셋 다 어려워보인다는 말씀에 공감합니다...

@hyp3rflow
Copy link

아.. 사실 이슈 찾는 중에 어려운거밖에 안보여서 그나마 견적 나오는걸 잡긴 한겁..니다... ㅠㅠ

@hyp3rflow
Copy link

Deno에서 이슈를 찾는 것에 실패해서 (어렵기도 하고 동기부여가 안되는게 컸습니다)
lottie 구현체를 rust로 만들어서 wasm 같은걸로 구워볼까 생각해서 https://github.com/zimond/lottie-rs 를 보고 있습니다.
내부에서 let-chain을 사용하는데 따로 버전을 명시해두지 않았고 stable에서 컴파일 에러가 나는 부분이 있어서 기여하려고 합니다.

@hyp3rflow
Copy link

일단 튜토리얼 쪽 문서가 outdated된 것 같아서 수정하는 PR을 올렸습니다. lottie-rs#4

@hyp3rflow
Copy link

lottie-rs#5를 올렸는데, 그냥 feature attribute를 한 줄 추가하고 nightly를 유지해도 괜찮겠다는 생각도 들었는데,
일단은 stable에서도 동작하게끔 사용하지 않는 feature attribute를 제거하고 let_chains를 걷어내는 방식으로 진행했습니다.

@hyp3rflow
Copy link

일단 lottie 쪽을 더 디깅할 여유가 안되어서 ㅠㅠ stc에 good first issue를 잔뜩 만들어주셔서 기여했습니다.

  • Conformance test들이 outdated라서 업데이트했구요: dudykr/stc@d121f20
  • 이슈를 하나 잡아서 해결했습니다. 간단히 설명하면 Variable이 아닌 것들에 assignment를 가했을 때 좀 더 친절한 에러가 나오도록 합니다: dudykr/stc@ba5c59c
  • Conformance test에 빠진 것이 있어서 채워넣었습니다: dudykr/stc@55439e8

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

No branches or pull requests

2 participants