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

Avoid class reference to imported function #578

Merged
merged 1 commit into from
Jan 25, 2016

Conversation

dbs
Copy link
Contributor

@dbs dbs commented Jan 17, 2016

We import the urljoin() function directly to support both Py2 and Py3, so the
reference to urlparse.urljoin() raises an exception. Don't do that!

Fixes 574.

Signed-off-by: Dan Scott dan@coffeecode.net

We import the urljoin() function directly to support both Py2 and Py3, so the
reference to urlparse.urljoin() raises an exception. Don't do that!

Fixes 574.

Signed-off-by: Dan Scott <dan@coffeecode.net>
@joernhees joernhees added bug Something isn't working enhancement New feature or request parsing Related to a parsing. labels Jan 25, 2016
@joernhees joernhees added this to the rdflib 4.2.2 milestone Jan 25, 2016
joernhees added a commit that referenced this pull request Jan 25, 2016
Avoid class reference to imported function
@joernhees joernhees merged commit bf1d0d9 into RDFLib:master Jan 25, 2016
@joernhees
Copy link
Member

thanks, we'll see what remains after the pyMicrodata cleanup though ;)

@joernhees
Copy link
Member

closed #574

@joernhees
Copy link
Member

@iherman did you sync this?

@iherman
Copy link
Contributor

iherman commented Jan 30, 2017

@joernhees if you mean whether my local copy is in sync with the current master (including your change in adding the csvw to the initial context, thanks for this!) then yes, I am in sync. I plan to update the initial context at the end of February, when some more Recommendations will be added to the mix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request parsing Related to a parsing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants