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

Feature/fat jar #62

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from
Open

Feature/fat jar #62

wants to merge 5 commits into from

Conversation

Madzi
Copy link
Member

@Madzi Madzi commented Sep 8, 2022

Запаковка всех библиотек в один jar файл.
Минимизирует количество файлов, которые необходимо иметь инсталлятору (или пользователю).

@realsonic
Copy link
Member

@Madzi привет!
Я сейчас собираю в один jar с помощью IDEA.

@realsonic
Copy link
Member

@Madzi почему-то в этом PR ещё переименование кнопок попало, не только работы по jar.

@Madzi
Copy link
Member Author

Madzi commented Sep 9, 2022

Привет.
К сожалению, у меня получились пул-реквесты так, что каждый следующий включает предыдущие.
С помощью Идеи, конечно, можно собирать, но лучше делать это системой сборки, которая будет делать это после каждого коммита, и сразу покажет если есть проблема.

Слак ещё живой ?

@realsonic
Copy link
Member

@Madzi Да, только что туда зашёл. Написал тебе там.

@realsonic
Copy link
Member

@Madzi Системой сборки - это отлично, только вот я как-то давно пытался собирать через Jenkins, и у меня не вышло из-за специфических библиотек. А у тебя вышло собирать через CI?

@realsonic realsonic self-requested a review October 28, 2023 11:54
@Madzi
Copy link
Member Author

Madzi commented Nov 21, 2023

@realsonic . Да, вот посмотри https://github.com/Madzi/ifml2/actions/workflows/mvn.yaml оно собирает все при каждом коммите

@Madzi
Copy link
Member Author

Madzi commented Nov 21, 2023

я заменил локальные библиотеки на те же самые библиотеки но из репозитория Maven. Так что и с Jenkins тоже не должно быть проблем

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

Successfully merging this pull request may close these issues.

2 participants