Skip to content

zadania: gildie #22

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

zadania: gildie #22

wants to merge 3 commits into from

Conversation

greg19
Copy link
Collaborator

@greg19 greg19 commented Jan 23, 2025

No description provided.

@greg19 greg19 requested review from lduraj and tonowak January 23, 2025 14:23
@tonowak
Copy link
Collaborator

tonowak commented Jan 23, 2025

Z reguły nie wstawia się plików >=1MB na git'a, teraz jak każdy będzie klonował repo to będzie musiał wszyściutko pobrać (nawet jeżeli nie trafi to ostatecznie na główny branch).

@tonowak
Copy link
Collaborator

tonowak commented Jan 23, 2025

Proponuję byś użył gitignore'a z templatki OI'owej.

@tonowak
Copy link
Collaborator

tonowak commented Jan 23, 2025

Nie wiem czy śledziłeś rozmowę z panią Joanną, tam podała informację, że prawdopodobnie musimy w skopiowanych zadaniach z OI/OIJ/OIG dodać informację o źródle.

@greg19
Copy link
Collaborator Author

greg19 commented Jan 23, 2025

Jak zrobię amend i push force to się te obiekty usuną, czy trzeba jeszcze większą magię gitową zrobić? Outy mogę usunąć, ale ta paczka nie ma ingena, więc z tym będzie problem.

@tonowak
Copy link
Collaborator

tonowak commented Jan 23, 2025

Oki, próbuj i sprawdź czy wyjdzie.

Hmm, skoro nie ma ingena, to może można by kombinować z git lfs?

@tonowak
Copy link
Collaborator

tonowak commented Jan 23, 2025

(w szczególności, PDFy też bym nie wrzucał)

@greg19
Copy link
Collaborator Author

greg19 commented Jan 24, 2025

Na razie wywaliłem PDF i outy oraz wzbogaciłem .gitignore.

Z tego co rozumiem to git lfs musi każdy sobie dodatkowo doinstalować. Biorąc pod uwagę powolną adaptację pre-commit, myślę że nie ma co blokować tego PR. Uważasz że ogólnie to ma sens? Mogę się tym zająć, tylko pytanie co powinienem priorytetyzować: MAP na stronie głównej, lekcje, git lfs, automatyzacja wgrywania paczek. Według mnie na razie dwa pierwsze.

@tonowak
Copy link
Collaborator

tonowak commented Jan 24, 2025

Myślę że to jest jak najbardziej OK na najbliższe dwa tygodnie, ale po tym trzeba będzie coś zrobić z in'ami w paczkach jeżeli chcemy tu mieć wiele paczek.

@tonowak
Copy link
Collaborator

tonowak commented Jan 24, 2025

Troszkę mnie github irytuje w tym sensie, że np. pokazuje, że ten PR ma 4M linii zmian.

@tonowak
Copy link
Collaborator

tonowak commented Jan 24, 2025

Spodziewam się, że jeżeli @lduraj będzie miał już trochę czasu, to będzie można się z nim zdwonić by na raz pokazać zarówno pre-commit, jak i git lfs. Właściwie to ten PR może sobie poczekać do tego czasu, a paczkę można już teraz wgrać.

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