List

SPOILERS DON'T READ IF YOU DON'T LIKE SPOILS

Cool things:
YODA!
YODA!
Snoke dying.
Holdo going kamikaze
Rey and Kylo teaming up
Rey's parents being deadbeat dads in a cigarette line
Luke using astral projection
Luke throwing the lightsaber off the cliff
Luke and Kylo having different perspectives on the origin of Kylo Ren story
Luke looking out at the twin suns
Everything about Luke's final confrontation
YODA!

Dumb things:
Flying Leia
Luke dying for no clear reason
Poe learning the same lesson 100 times about not being trigger happy
Captain Phasma for the second movie in a row
DJs awful stutter (and character)
Animal cruelty message.. because... idk
Some of the space battles were boring video game cutscenes tbh
Poe Rey and Finn have only all JUST MET EACH OTHER and it's the end of the SECOND MOVIE!
Holdo not telling Poe what the plan was JUST so he could learn the same lesson AGAIN
Humor didn't work as good as it did in TFA