REL: v1.1.4

[MateusStano]

[v1.1.4] - 2023-12-07

Fixed

• DOC: Change from % to ! in the first cell to run properly in Colab. #496
• FIX: changes Generic Motor exhaust velocity to cached property #497

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (8d6f244) 70.95% compared to head (0c49c0b) 70.92%.
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #503      +/-   ##
==========================================
- Coverage   70.95%   70.92%   -0.04%     
==========================================
  Files          55       55              
  Lines        9262     9262              
==========================================
- Hits         6572     6569       -3     
- Misses       2690     2693       +3     

Flag	Coverage Δ
unittests	70.92% <100.00%> (-0.04%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Gui-FernandesBR]

The CHANGELOG seems messy.
I'm going to take a look at this rn

[Gui-FernandesBR]

This line is not necessary for changelog. I'm going to update the changelog description to avoid future mistakes.

PLease don't accept suggestion

Suggested change

- DOC: Change from % to ! in the first cell to run properly in Colab. [#496](https://github.com/RocketPy-Team/RocketPy/pull/496)

[Gui-FernandesBR]

This Looks Good To Me

[giovaniceotto]

I am not comfortable with changes made to the changelog file description. A release PR is not the place to make them, so I request they are removed for now. Sorry for being so pedantic here, but I believe this is important. In Brazil, such changes are referred as "jabutis" and they can be quite problematic in the long run since the discussion of these changes are often overlooked and shadowed by more important things.

Although not important here, in my opinion, the changelog file is not a release notes meant to please only end users. We already have the official release notes for that. The changelog is meant for all involved in the project to keep track of what is changing.

*Just to be clear, I am not against changes to the changelog, I am against changes to the scope of the changelog file, as a tag along in this PR.

[Gui-FernandesBR]

I am not comfortable with changes made to the changelog file description. A release PR is not the place to make them, so I request they are removed for now. Sorry for being so pedantic here, but I believe this is important. In Brazil, such changes are referred as "jabutis" and they can be quite problematic in the long run since the discussion of these changes are often overlooked and shadowed by more important things.

Although not important here, in my opinion, the changelog file is not a release notes meant to please only end users. We already have the official release notes for that. The changelog is meant for all involved in the project to keep track of what is changing.

*Just to be clear, I am not against changes to the changelog, I am against changes to the scope of the changelog file, as a tag along in this PR.

Thank you for submitting a review, @giovaniceotto ,

I reverted the previous commit as suggested.

[Gui-FernandesBR]

Thank you @MateusStano