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

gh-97709: Included newline separator in Mandelbrot set #97737

Merged
merged 2 commits into from
Oct 4, 2022
Merged

gh-97709: Included newline separator in Mandelbrot set #97737

merged 2 commits into from
Oct 4, 2022

Conversation

matheusja
Copy link
Contributor

@matheusja matheusja commented Oct 2, 2022 •
edited by bedevere-bot

Now the Mandelbrot set one-liner example on separates the lines with a '\n' character, instead of relying on line wrapping to separate them.
Also, since the code is now modified, it's not the one "due to Ulf Bartelt", but "adapted from Ult Bartelt".

@matheusja
Included newline separator in Mandelbrot set
d74e4d5 
Now the Mandelbrot set one-liner example on separates the lines with a '\n' character.
@bedevere-bot bedevere-bot added awaiting review docs Documentation in the Doc dir skip news labels Oct 2, 2022
gpshead
gpshead approved these changes Oct 4, 2022
View reviewed changes
Copy link
Member

@gpshead gpshead left a comment

Choose a reason for hiding this comment

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

heh, yes, without that the code produces a mess.

@gpshead gpshead added the sprint label Oct 4, 2022
@gpshead gpshead merged commit 4980260 into python:main Oct 4, 2022
@gpshead gpshead added needs backport to 3.10 only security fixes needs backport to 3.11 only security fixes labels Oct 4, 2022
@miss-islington
Copy link
Contributor

Thanks @matheusja for the PR, and @gpshead for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Thanks @matheusja for the PR, and @gpshead for merging it 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-97823 is a backport of this pull request to the 3.10 branch.

@bedevere-bot bedevere-bot removed needs backport to 3.10 only security fixes needs backport to 3.11 only security fixes labels Oct 4, 2022
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 4, 2022
@matheusja @miss-islington
pythongh-97709: Included newline separator in Mandelbrot set (pythonG…
69fcee9 
…H-97737)

Included newline separator in Mandelbrot set

Now the Mandelbrot set one-liner example on separates the lines with a '\n' character.
(cherry picked from commit 4980260)

Co-authored-by: matheusja <matheusjahnke@hotmail.com>
@bedevere-bot
Copy link

GH-97824 is a backport of this pull request to the 3.11 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 4, 2022
@matheusja @miss-islington
pythongh-97709: Included newline separator in Mandelbrot set (pythonG…
98c3dac 
…H-97737)

Included newline separator in Mandelbrot set

Now the Mandelbrot set one-liner example on separates the lines with a '\n' character.
(cherry picked from commit 4980260)

Co-authored-by: matheusja <matheusjahnke@hotmail.com>
miss-islington added a commit that referenced this pull request Oct 4, 2022
@miss-islington @matheusja
gh-97709: Included newline separator in Mandelbrot set (GH-97737)
c62be23 
Included newline separator in Mandelbrot set

Now the Mandelbrot set one-liner example on separates the lines with a '\n' character.
(cherry picked from commit 4980260)

Co-authored-by: matheusja <matheusjahnke@hotmail.com>
JelleZijlstra pushed a commit that referenced this pull request Oct 4, 2022
@miss-islington @matheusja
[3.10] gh-97709: Included newline separator in Mandelbrot set (GH-97737…
19ed29e 
…) (#97823)

Included newline separator in Mandelbrot set

Now the Mandelbrot set one-liner example on separates the lines with a '\n' character.
(cherry picked from commit 4980260)

Co-authored-by: matheusja <matheusjahnke@hotmail.com>
pablogsal pushed a commit that referenced this pull request Oct 22, 2022
@miss-islington @matheusja @pablogsal
gh-97709: Included newline separator in Mandelbrot set (GH-97737)
4dda0c8 
Included newline separator in Mandelbrot set

Now the Mandelbrot set one-liner example on separates the lines with a '\n' character.
(cherry picked from commit 4980260)

Co-authored-by: matheusja <matheusjahnke@hotmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gpshead gpshead gpshead approved these changes

Assignees
No one assigned
Labels
docs Documentation in the Doc dir skip news sprint
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@matheusja @miss-islington @bedevere-bot @gpshead