Skip to content

Broken link in isomorphism documentation#6296

Merged
MridulS merged 5 commits intonetworkx:mainfrom
paulitapb:broken_link_isomorphism
Jan 3, 2023
Merged

Broken link in isomorphism documentation#6296
MridulS merged 5 commits intonetworkx:mainfrom
paulitapb:broken_link_isomorphism

Conversation

@paulitapb
Copy link
Copy Markdown
Member

As I was reading the documentation I found that the link in the reference section of is_isomorphic is broken. I replaced the current link with a new version but I'm not sure if it's the right one. The paper seems to be the same. Maybe the citation format is not right.

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.

Looks like this is change from a wrong branch?

The changes aren't for the isomorphism docs.

@paulitapb
Copy link
Copy Markdown
Member Author

Looks like this is change from a wrong branch?

The changes aren't for the isomorphism docs.

Yes, It seems this is on my main branch. I will fix that.

@paulitapb
Copy link
Copy Markdown
Member Author

Looks like this is change from a wrong branch?

The changes aren't for the isomorphism docs.
Now the right changes are here

Copy link
Copy Markdown
Contributor

@rossbar rossbar left a comment

Choose a reason for hiding this comment

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

The new link works and links to the correct doc so LGTM, thanks @paulitapb !

@MridulS MridulS merged commit 83c22da into networkx:main Jan 3, 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
* Add warning in community doc

* Add new link to the doc
dschult pushed a commit to BrunoBaldissera/networkx that referenced this pull request Oct 23, 2023
* Add warning in community doc

* Add new link to the doc
cvanelteren pushed a commit to cvanelteren/networkx that referenced this pull request Apr 22, 2024
* Add warning in community doc

* Add new link to the doc
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants