Skip to content

[3.6] bpo-33640, uuid.UUID doc: document endian of bytes parameter (GH-7263)#7386

Merged
miss-islington merged 1 commit into
python:3.6from
miss-islington:backport-b75ec08-3.6
Jun 4, 2018
Merged

[3.6] bpo-33640, uuid.UUID doc: document endian of bytes parameter (GH-7263)#7386
miss-islington merged 1 commit into
python:3.6from
miss-islington:backport-b75ec08-3.6

Conversation

@miss-islington

@miss-islington miss-islington commented Jun 4, 2018

Copy link
Copy Markdown
Contributor

The bytes parameter uses big endian.
(cherry picked from commit b75ec08)

Co-authored-by: Farhaan Bukhsh farhaan.bukhsh@gmail.com

https://bugs.python.org/issue33640

…H-7263)

The bytes parameter uses big endian.
(cherry picked from commit b75ec08)

Co-authored-by: Farhaan Bukhsh <farhaan.bukhsh@gmail.com>

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM, good bot.

@miss-islington

Copy link
Copy Markdown
Contributor Author

@farhaanbukhsh and @vstinner: Backport status check is done, and it's a success ✅ .

@Mariatta

Mariatta commented Jun 4, 2018

Copy link
Copy Markdown
Member

I've opened an issue to investigate why miss-islington did not automerge this PR: python/miss-islington#94

@Mariatta

Mariatta commented Jun 4, 2018

Copy link
Copy Markdown
Member

The PR was not automerged because at that time, appveyor was a required status check, but it did not run.
Appveyor status check has been disabled now, so I'll get miss-islington to merge this.

@miss-islington

Copy link
Copy Markdown
Contributor Author

@farhaanbukhsh and @vstinner: Backport status check is done, and it's a success ✅ .

@miss-islington
miss-islington merged commit 6a43d5f into python:3.6 Jun 4, 2018
@miss-islington

Copy link
Copy Markdown
Contributor Author

Thanks!

@miss-islington
miss-islington deleted the backport-b75ec08-3.6 branch June 4, 2018 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants