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

Добавить инструкцию по контрибьютингу #15

Open
Dennis-Nedry-From-Jurassic-Park opened this issue Jul 27, 2023 · 6 comments

Comments

@Dennis-Nedry-From-Jurassic-Park
  1. pull-proto
  2. compile-proto
  3. добавляем функционал (допустим positionsStream)
  4. lint & test
  5. заводим PR

примерно так @vitalets ?

@vitalets
Copy link
Owner

В целом да.
4. lint & test автоматически запустится на пре-коммит / пре-пуш

@Dennis-Nedry-From-Jurassic-Park
Copy link
Author

Dennis-Nedry-From-Jurassic-Park commented Aug 11, 2023

@vitalets случаем нет скрипта как сбилдить под виндой контракты ?,
запустил скрипт под оболочкой Linux (wsl2)
контракты сбилдило, но сломалось на этом месте

bash ./scripts/compile-proto.sh     

sed: can't read s|import \* as _m0 from "protobufjs/minimal";|import _m0 from "protobufjs/minimal\.js";|g: 
No such file or directory
sed: can't read s|from "\./([^"]+)";|from "\./\1\.js";|g:
No such file or directory

в generated эти файлы

image

@vitalets
Copy link
Owner

По-моему для linux у sed чуть другие аргументы, вот тут обсуждение: https://unix.stackexchange.com/questions/401905/bsd-sed-vs-gnu-sed-and-i

Вообще вместо sed можно нодовский пакет заюзать (типа этого), чтобы не париться с os совместимостью.

@Dennis-Nedry-From-Jurassic-Park
Copy link
Author

@vitalets если вкратце, я должен обойти контракты и заменить
import * as _m0 from "protobufjs/minimal";
на
import _m0 from "protobufjs/minimal.js";

верно ?

@Dennis-Nedry-From-Jurassic-Park
Copy link
Author

вот эту строку что-то не разобрал
s|from "./([^"]+)";|from "./\1.js";|g

а ну, добавляется .js просто по идее

@vitalets
Copy link
Owner

@vitalets если вкратце, я должен обойти контракты и заменить import * as _m0 from "protobufjs/minimal"; на import _m0 from "protobufjs/minimal.js";

верно ?

Да!

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