my big complaint is the vilains. This is usually Marvels weakness. The Skrulls should have been genuinely scary in and body snatches kinda way. Instead they were just there. Oh wait, they are just galactic refugees now. The Kree characters were one-dimensional and the more they didn't say Yon-Rogg's name, the more I knew who it was. In fact, I don't know if his name was ever mentioned till the credits.
I thought Brie was good as Carol. In a movie where everything verged on being ridiculous, she gave it a grounding force. It will be interesting to see how Carol reacted to the past 20+ years and the snap.
Sam Jackson was pretty awesome. More light-hearted, less burdened by the weight of the world.
I also really liked Lashana Lynch as Maria. While her role was limited till the last third of the movie or so, I thought she knocked it out of the park during her reunion with Carol. She also kicked ass being a pilot in that chase scene.
The effects were mostly good. They did look a little off in a few scenes, especially the ones where Carol was in her full suit including the helmet. The practical effects were also really good and I liked the car/train scene a lot.