Skip to content

Fix typo of URL of sbol data model specification #425

Merged
tcmitchell merged 7 commits intoSynBioDex:mainfrom
Omarelsherif010:URL-of-SBOL-data-model-specification-
Mar 27, 2023
Merged

Fix typo of URL of sbol data model specification #425
tcmitchell merged 7 commits intoSynBioDex:mainfrom
Omarelsherif010:URL-of-SBOL-data-model-specification-

Conversation

@Omarelsherif010
Copy link
Copy Markdown
Contributor

Inside constant.py file the URL of SBOL data model specification is not correct.

I am not sure if it is better to change the URL itself to look like the one written in file or change the one in file for now until next update.

currently this make the URL inside the file send the user to SBOL data model specification website.

fixes #424

Copy link
Copy Markdown
Collaborator

@tcmitchell tcmitchell left a comment

Choose a reason for hiding this comment

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

Please remove the changes to .gitignore, or explain why they are necessary.

Comment thread .gitignore Outdated
dist/
# Emacs backup files
*~
test/SBOLTestSuite
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why is this being ignored? Is this part of the URL change? I don't think this is right.

@tcmitchell
Copy link
Copy Markdown
Collaborator

This is a good catch, thanks for updating the URL. The sbolstandard web site was migrated last year and the URL must have changed as part of that migration. You are correct that the URL should be updated in constants.py to match the new URL at sbolstandard.org

Copy link
Copy Markdown
Collaborator

@tcmitchell tcmitchell left a comment

Choose a reason for hiding this comment

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

Please revert .gitignore since there are no changes other than whitespace.

Comment thread .gitignore Outdated
Copy link
Copy Markdown
Collaborator

@tcmitchell tcmitchell left a comment

Choose a reason for hiding this comment

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

Please revert .gitignore since there are no changes other than whitespace. You'll know you have reverted it properly when it no longer appears in this pull request.

.vscode folder was created automatically when I run unittest for specific branch.

I found that this folder was unnecessary, so I have deleted it and removed from gitignore file
@Omarelsherif010
Copy link
Copy Markdown
Contributor Author

Finally reverted the file

Now, the .gitignore file in my branch is the same as the one on the main

@tcmitchell tcmitchell merged commit 3812ff1 into SynBioDex:main Mar 27, 2023
@Omarelsherif010 Omarelsherif010 changed the title fix typo of URL of sbol data model specification Fix typo of URL of sbol data model specification Mar 28, 2023
@tcmitchell tcmitchell added this to the 1.1 milestone Apr 11, 2023
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.

fix typo in URL of SBOL data model specification

2 participants