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.

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

ENH: Rename complex to simplnx #801

Merged

Conversation

imikejackson
Copy link
Contributor

The complex library is renamed to "SIMPLNX"

@joeykleingers joeykleingers self-requested a review December 20, 2023 16:19
Signed-off-by: Michael Jackson <mike.jackson@bluequartz.net>
@imikejackson imikejackson force-pushed the feature/simplnx_rename branch from 6c763a0 to 6c8b5d9 Compare December 21, 2023 14:12
Copy link
Contributor

Choose a reason for hiding this comment

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

Not an actual problem just a style warning:
Just a heads up on this file the all-contributors bot will post links with the uppercase BlueQuartzSoftware, since that is the way the URL is distributed. That being said I checked the github docs and organization names are case insensitive so it shouldn't matter either way (and URLS are obviously case insensitive when making requests outside the local OS filesystem). All this is to say that it isn't breaking and lowercase URLs conforms to typical conventions, but I fear uniformity may be broken when we add new contributors, since this is a niche styling decision that will likely not be considered/remembered when reviewing the new contributors PR.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'll get this cleaned up on the back side of this conversion.

@imikejackson imikejackson merged commit a5b7305 into BlueQuartzSoftware:develop Dec 22, 2023
7 checks passed
@imikejackson imikejackson deleted the feature/simplnx_rename branch December 22, 2023 16:30
imikejackson added a commit that referenced this pull request Dec 22, 2023
The previous repo would have been at https://www.github.com/bluequartzsoftware/complex

Signed-off-by: Michael Jackson <mike.jackson@bluequartz.net>
imikejackson added a commit that referenced this pull request Dec 22, 2023
The previous repo would have been at https://www.github.com/bluequartzsoftware/complex

Signed-off-by: Michael Jackson <mike.jackson@bluequartz.net>
imikejackson added a commit to imikejackson/simplnx that referenced this pull request Oct 20, 2024
The previous repo would have been at https://www.github.com/bluequartzsoftware/complex

Signed-off-by: Michael Jackson <mike.jackson@bluequartz.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants