Skip to content

fix whitespace issue in test_internet_as_graphs#6324

Merged
dschult merged 1 commit intonetworkx:mainfrom
danieleades:whitespace
Jan 4, 2023
Merged

fix whitespace issue in test_internet_as_graphs#6324
dschult merged 1 commit intonetworkx:mainfrom
danieleades:whitespace

Conversation

@danieleades
Copy link
Copy Markdown
Contributor

@danieleades danieleades commented Jan 4, 2023

the original strings are incorrectly split. I suspect rather than

                raise ValueError(
                    "Inconsistent data in the graph\
                        node attributes"
                )

the intention was probably to have

                raise ValueError(
                    "Inconsistent data in the graph"
                    " node attributes"
                )

but it all fits on one line anyway

MridulS
MridulS approved these changes Jan 4, 2023
View reviewed changes
Copy link
Copy Markdown
Member

@MridulS MridulS left a comment

Choose a reason for hiding this comment

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

Thanks @danieleades !

@dschult dschult merged commit 814b295 into networkx:main Jan 4, 2023
@danieleades danieleades deleted the whitespace branch January 4, 2023 14:36
@dschult dschult changed the title fix whitespace issue in 'networkx.generators.tests.test_internet_as_g… fix whitespace issue in test_internet_as_graphs Jan 4, 2023
@jarrodmillman jarrodmillman added this to the networkx-3.0 milestone Jan 4, 2023
MridulS pushed a commit to MridulS/networkx that referenced this pull request Feb 4, 2023
@danieleades @MridulS
fix whitespace issue in 'networkx.generators.tests.test_internet_as_g… (
bc916cc 
networkx#6324)

fix whitespace issue in 'networkx.generators.tests.test_internet_as_graphs'

Co-authored-by: daniel.eades <daniel.eades@hotmail.com>
dschult pushed a commit to BrunoBaldissera/networkx that referenced this pull request Oct 23, 2023
@danieleades @dschult
fix whitespace issue in 'networkx.generators.tests.test_internet_as_g… (
2056012 
networkx#6324)

fix whitespace issue in 'networkx.generators.tests.test_internet_as_graphs'

Co-authored-by: daniel.eades <daniel.eades@hotmail.com>
cvanelteren pushed a commit to cvanelteren/networkx that referenced this pull request Apr 22, 2024
@danieleades @cvanelteren
fix whitespace issue in 'networkx.generators.tests.test_internet_as_g… (
3412e1b 
networkx#6324)

fix whitespace issue in 'networkx.generators.tests.test_internet_as_graphs'

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

Reviewers

@dschult dschult dschult approved these changes

@MridulS MridulS MridulS approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

networkx-3.0

Development

Successfully merging this pull request may close these issues.

4 participants

@danieleades @dschult @MridulS @jarrodmillman