Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

make merge#1300

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
JulienPalard merged 3 commits intopython:3.8fromJulienPalard:make-merge
May 24, 2020
Merged

make merge#1300

JulienPalard merged 3 commits intopython:3.8fromJulienPalard:make-merge
May 24, 2020

Conversation

JulienPalard
Copy link
Member

Je n'ai pas touché au fuzzy, juste fait un make merge.

@deronnax
Copy link
Collaborator

Qui êtes vous, monsieur ? Comment avez-vous eu accès au dépôt ?

christopheNan reacted with laugh emoji

Comment on lines +1561 to +1562
"Lève un :ref:`auditing event <auditing>` ``builtins.breakpoint`` avec "
"l'argument ``breakpointhook``."
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
"Lève un :ref:`auditing event <auditing>` ``builtins.breakpoint`` avec"
"l'argument ``breakpointhook``."
"Lève un :ref:`évènement d'audit <auditing>` ``builtins.input/result` avec"
"l'argument ``result``."

Copy link
Collaborator

Choose a reason for hiding this comment

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

c'est pas lui qui casse la CI, hein

Copy link
Contributor

Choose a reason for hiding this comment

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

Exact, j'ai lu trop vite.

@awecx
Copy link
Contributor

library/idle.po:1712:appellant
library/sys.po:558:breakpointhook

D'où ça vient ça ?

Co-authored-by: Antoine <43954001+awecx@users.noreply.github.com>
#: ../Doc/library/sys.rst:346
#, fuzzy
msgid "__unraisablehook__"
msgstr "__breakpointhook__"
Copy link
Collaborator

Choose a reason for hiding this comment

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

c'est lui. Et alors c'est très bizarre parce que son pote juste au-dessus___breakpointhook__ lui semble passer le pospell.

Copy link
MemberAuthor

Choose a reason for hiding this comment

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

Celui juste au dessus passe car pospell ignore quand msgid == msgstr.

@JulienPalard
Copy link
MemberAuthor

library/idle.po:1712:appellant
library/sys.po:558:breakpointhook

D'où ça vient ça ?

$ git blame library/idle.po | grep appellant1bf9a2fa0 (fsetton              2020-05-23 10:01:36 +0200 1716) "Lorsque l'utilisateur lève `SystemExit` directement ou en appellant ``sys."

et le fuzzylibrary/sys.po:558:breakpointhook ça vient du make merge, c'estsphinx-build -b gettext qui se permet de rajouter qq fuzzy au cas où ça aide.

@JulienPalard
Copy link
MemberAuthor

Intéressant pospell qui bloque sur dunder breakpointhook :(

@JulienPalard
Copy link
MemberAuthor

Je tente avec des double backticks.

@deronnax
Copy link
Collaborator

deronnax commentedMay 24, 2020
edited
Loading

Autre truc intriguant : la PR qui a introduit l'erreur (la#1175) est bien verte, la CI lui a donné le go, malgré le "appellant" invalide. Ça signifie qu'il y a un problème avec la CI (soit la détection du Makefile des fichier à analyser, soit pospell)

@JulienPalard
Copy link
MemberAuthor

JulienPalard commentedMay 24, 2020
edited
Loading

Louche, je la voit bien rouge sur travis :https://travis-ci.org/github/python/python-docs-fr/builds/690293347 :

$ if [ -n "$CHANGED_FILES" ]; then pospell -p dict -l fr_FR $CHANGED_FILES; filibrary/idle.po:1705:appellantThe command "if [ -n "$CHANGED_FILES" ]; then pospell -p dict -l fr_FR $CHANGED_FILES; fi" exited with 255.

@JulienPalardJulienPalard merged commit00a0972 intopython:3.8May 24, 2020
@deronnax
Copy link
Collaborator

Le commit de merge est rouge mais pas la PR. Je ne comprends pas comment cela peut être possible.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@awecxawecxawecx left review comments

@deronnaxderonnaxderonnax approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@JulienPalard@deronnax@awecx

[8]ページ先頭

©2009-2025 Movatter.jp