Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(build): wrong file extension #543

Merged
merged 2 commits into from
Dec 26, 2024

Conversation

Gabriel29306
Copy link
Contributor

🚀 Nouvelle Pull Request

Checklist d'avant pull request

Veuillez cocher toutes les cases applicables en remplaçant [ ] par [x].

  • Vous avez testé de build le projet avec vos modifications et ce build a réussi
  • Cette pull request n'est pas un duplicata d'une autre
  • Cette pull request est prête à être revue (review) et fusionnée (merge)
  • Les détails des changements ont été décrits ci-dessous
  • Cette pull-request n'est pas une "breaking-change" (des modifications qui vont entraîner la modification du fonctionnement de certaines fonctionnalités déjà existantes)

Changelogs proposés

Sous le format png au lieu de jpg, l'image d'Alise n'est pas bien traité par AAPT lors de la compilation pour Android.

https://github.com/Gabriel29306/PapillonV7/actions/runs/12473686118/job/34814586374

@raphckrman
Copy link
Contributor

@Gabriel29306
Copy link
Contributor Author

My bad

@Gabriel29306
Copy link
Contributor Author

Hm, il y a d'autres bêtises (et pas que de typage) qu'il faudra régler avec une autre PR

@raphckrman
Copy link
Contributor

oups sur Alise?

Copy link
Collaborator

@oriionn oriionn left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@Clmnnt Clmnnt left a comment

Choose a reason for hiding this comment

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

LGTM 💯💯

@Gabriel29306
Copy link
Contributor Author

oups sur Alise?

Non, sur d'autres parties de l'app

Copy link
Contributor

@JyhuKo JyhuKo left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@raphckrman raphckrman left a comment

Choose a reason for hiding this comment

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

LGTM

@Gabriel29306
Copy link
Contributor Author

Toutes les erreurs TS/ES Lint sont provoquées par d'autres PR. Les changements ici n'altèrent en rien le comportement de l'application ainsi que rien dans les erreurs TS/ES Lint.

Copy link
Contributor

@Kgeek33 Kgeek33 left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@ecnivtwelve ecnivtwelve merged commit 8824976 into PapillonApp:main Dec 26, 2024
1 check failed
@Gabriel29306 Gabriel29306 deleted the fixBuild/alise branch December 27, 2024 15:35
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.

7 participants