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.

Sign up for GitHub

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

Jump to bottom

Tech : assertSnapshotQueries() avec des annotations chainées #4777

Merged
merged 2 commits into from
Oct 3, 2024
Merged

Tech : assertSnapshotQueries() avec des annotations chainées #4777

merged 2 commits into from
Oct 3, 2024

Conversation

rsebille
Copy link
Contributor

@rsebille rsebille added the no-changelog Ne doit pas figurer dans le journal des changements. label Sep 19, 2024
@rsebille rsebille self-assigned this Sep 19, 2024
@rsebille rsebille mentioned this pull request Sep 19, 2024
Merged
@xavfernandez xavfernandez force-pushed the rsebille/multiple-annotate-unstable-order branch from eb98c54 to 5aacfe9 Compare October 3, 2024 09:02
@xavfernandez xavfernandez marked this pull request as ready for review October 3, 2024 09:02
@xavfernandez xavfernandez self-assigned this Oct 3, 2024
@xavfernandez
Copy link
Contributor

@rsebille Je ne peux pas te mettre reviewer de ta propre PR mais ton avis m'intéresse fortement :)

rsebille
rsebille commented Oct 3, 2024
View reviewed changes
Copy link
Contributor Author

@rsebille rsebille left a comment

Choose a reason for hiding this comment

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

Je peux pas tamponner mais ça me semble très bien 😁.
Le fait qu'on sache déjà que ça va casser dans le futur n'est pas très "rassurant" mais soit on regardera pour adapter à ce moment là soit on reviendra à un mode dégradé et on aura tout de même bénéficier de la fonction pendant ce temps là.

@xavfernandez
Copy link
Contributor

Le fait qu'on sache déjà que ça va casser dans le futur n'est pas très "rassurant" mais soit on regardera pour adapter à ce moment là soit on reviendra à un mode dégradé et on aura tout de même bénéficier de la fonction pendant ce temps là.

Alors le patch va arrêter de faire ce qu'on veut (vu que Query.annotation_select_mask devient un set alors que c'est une list en 5.0), mais j'ai un petit espoir également que le soucis soit corrigé par les modifs dans le fonctionnement de tout cela 🤞

@francoisfreitag
Copy link
Contributor

Au pire, on trichera avec un ordered set ;)

@xavfernandez xavfernandez added this pull request to the merge queue Oct 3, 2024
Merged via the queue into master with commit 5377608 Oct 3, 2024
11 checks passed
@xavfernandez xavfernandez deleted the rsebille/multiple-annotate-unstable-order branch October 3, 2024 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@francoisfreitag francoisfreitag francoisfreitag approved these changes

Assignees

@xavfernandez xavfernandez

@rsebille rsebille

Labels
no-changelog Ne doit pas figurer dans le journal des changements.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@rsebille @xavfernandez @francoisfreitag