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

Feature/improve namespace #20

Merged
merged 9 commits into from
May 27, 2020
Merged

Feature/improve namespace #20

merged 9 commits into from
May 27, 2020

Conversation

4lm
Copy link
Collaborator

@4lm 4lm commented Dec 5, 2019

Hi all,

please review this PR it changes the namespace from "metadata" to "oemetadata". Concerning issue #15

Beware, 2 of 16 tests are failing, because I didn't change the repo name "yet". Will do if this PR gets accepted ...

Thanks!

@@ -1,15 +0,0 @@
def test_if_schema_json_loads_successfully():
Copy link
Contributor

Choose a reason for hiding this comment

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

Why did you remove these files in a commit with the title "Rename metadata to oemetadata" and not in a subsequent commit?

Copy link
Contributor

Choose a reason for hiding this comment

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

ok I understood, renaming the file ...

Copy link
Member

@Ludee Ludee left a comment

Choose a reason for hiding this comment

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

Do we also want to rename the repo?

@jh-RLI
Copy link
Contributor

jh-RLI commented Apr 28, 2020

@Ludee I think we should rename the repo to show a clear naming convention for package and repo

@jh-RLI
Copy link
Contributor

jh-RLI commented Apr 28, 2020

@Ludee Unfortunately a pytest still causes me problems. For some reason the assertion for the strings "$id" and "$schema" from the schema.json fails. Actually only two strings are compared here. I have tested everything so far and could get a successful run but a second run failed again. I am not sure where the error comes from.

@jh-RLI
Copy link
Contributor

jh-RLI commented Apr 28, 2020

Got it there was already "oemetadata" used as repo name in the URL string :)

@jh-RLI
Copy link
Contributor

jh-RLI commented Apr 29, 2020

@Ludee Now all tests have been passed and we are ready to merge. But you're gonna have to rename the repo to "oemetadata" because I can't do that.

@jh-RLI jh-RLI merged commit f7a09d6 into develop May 27, 2020
@jh-RLI jh-RLI deleted the feature/improve-namespace branch January 18, 2021 13:17
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.

Change repo/package/module namespace to oemetadata Tests to validate schema against meta schema fail.
4 participants