Skip to content

Conversation

@vjousse
Copy link
Collaborator

@vjousse vjousse commented Oct 7, 2024

πŸ”§ Problem

Following the merge of #757, airTransportRatio field is not needed anymore in exemples.

Notion card

🍰 Solution

Remove the field from the examples file.

🏝️ How to test

Tests should still be green and scores should still be the same.
The test scenario described in this Notion card should still work.

@vjousse vjousse self-assigned this Oct 7, 2024
@vjousse vjousse requested a review from n1k0 October 7, 2024 08:40
Copy link
Member

@n1k0 n1k0 left a comment

Choose a reason for hiding this comment

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

LGTM

@vjousse vjousse merged commit ad762ee into master Oct 8, 2024
@vjousse vjousse deleted the fix/remove-air-transport-ratio-from-examples branch October 8, 2024 08:59
vjousse pushed a commit that referenced this pull request Oct 10, 2024
## [2.4.0](https://github.com/MTES-MCT/ecobalyse/compare/v2.3.0..v2.4.0)
(2024-10-10)



### πŸš€ Features

- Introduce first version of object interface
([#756](#756))

### πŸͺ² Bug Fixes

- Sync food ([#759](#759))
- Don't hide version information on staging
([#778](#778))
- Reset physical durablility in regulatory mode
([#786](#786))
- *(api,food)* Nullable fields weren't nullable anymore.
([#789](#789))

### 🚜 Refactor

- Small textile explorer improvements
([#773](#773))

### βš™οΈ Miscellaneous Tasks

- Don't download draft releases
([#771](#771))
- Remove `airTransportRatio` from examples
([#785](#785))
- Cleanup package-lock.json.
([#787](#787))
- Use builtin python action cache for pipenv
([#796](#796))
- Improve changelog by using `git-cliff`
([#768](#768))

<!-- generated by git-cliff -->

Signed-off-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

3 participants