-
Notifications
You must be signed in to change notification settings - Fork 5
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
Extensions building refactoring #62
Comments
В 9470020 вынес общий класс для двух билдеров. Для файрфокса должен работать, а для хрома явно не хватает файлов (например, manifest.json). Чтобы все было совсем кошерно, нужно оставить директорию extensions в качестве шаблона, так чтобы она была целиком под VCS, а билд-скрипт должен создавать результат компиляции где-то в папке result (которая, соответственно, будет в ignore). |
Там всё есть, на виндах собирается, значит где-то проблема с большими/маленькими буквами. |
Мне кажется, что что-то есть у тебя локально, чего нет в VCS. https://github.com/crimaniak/dirty-on-steroids/tree/dev/extensions/chrome |
ага, там в gitignore оказался manifest и иконка. только там ещё надо сертификат, и его нельзя шарить вот так открыто, если мы хотим централизованно делать обновления. может через инбокс раздать? |
Общий issue для всего, касающегося улучшения процесса сборки extensions.
На данный момент нужно:
The text was updated successfully, but these errors were encountered: