Skip to content

09 Seznam unikátních prvků - #39

Merged
Rebarbora6 merged 1 commit into
mainfrom
ak-09-unique-elements
Oct 15, 2025
Merged

09 Seznam unikátních prvků#39
Rebarbora6 merged 1 commit into
mainfrom
ak-09-unique-elements

Conversation

@Rebarbora6

Copy link
Copy Markdown
Collaborator

No description provided.

@Rebarbora6 Rebarbora6 closed this Oct 8, 2025
@Rebarbora6
Rebarbora6 deleted the ak-09-unique-elements branch October 8, 2025 12:47
@Rebarbora6 Rebarbora6 reopened this Oct 8, 2025
@Rebarbora6 Rebarbora6 linked an issue Oct 8, 2025 that may be closed by this pull request
Closed

@dslavicek dslavicek left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prosím nepoužívej podtržítko jako název proměnné. Podtržítko ve for cyklu je vhodné používat pouze když řídící proměnnou for cyklu nikde dál nevyužíváš.

Dále je podle zadání potřeba před review spojit všechny commity do jednoho.

Spojení commitu jde udělat přes git rebase -i <hash commitu předcházejícího všem commitům, které chceš spojovat> (špičatý závorky tam nepatří, těma jen ukazuju kde má být hash). Následně se ti otevře textový editor s commitama a slovama pick u commitů. U commitu který chceš připojit k předchozímu přepíšeš pick na squash, uložíš a zavřeš editor. Potom se ti znova otevře textový editor, tentokrát bude chtít novou commit message. Napíšeš, uložíš, zavřeš a máš sqashnuto. Pushovat je pak potřeba s --force, protože sqashováním měníš historii.

Kdyžtak si řekni o pomoc někomu z koučů před lekcí.

@Rebarbora6
Rebarbora6 force-pushed the ak-09-unique-elements branch from 954ff0d to 8eaffcc Compare October 8, 2025 21:02
@Rebarbora6

Copy link
Copy Markdown
Collaborator Author

Díky za zpětnou vazbu. Myslím, že nakonec dobré, jen mi nešlo squashnout 3 commity do 1 a musela jsem to dělat nadvakrát. To je běžná praxe, nebo to lze udělat i elegantněji v jednom běhu?

@dslavicek

Copy link
Copy Markdown
Collaborator

Díky za zpětnou vazbu. Myslím, že nakonec dobré, jen mi nešlo squashnout 3 commity do 1 a musela jsem to dělat nadvakrát. To je běžná praxe, nebo to lze udělat i elegantněji v jednom běhu?

Jde to udělat v jednom běhu. U nejstaršího commitu necháš pick u všech dalších přepíšeš na squash.

@dslavicek dslavicek left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

V pořádku :-)

@Rebarbora6
Rebarbora6 merged commit f0d749c into main Oct 15, 2025
3 checks passed
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.

09

2 participants