Skip to content
This repository has been archived by the owner on Mar 28, 2019. It is now read-only.

Refactor the imports - Refs #34 #277

Closed
wants to merge 1 commit into from

Conversation

hiromipaw
Copy link
Contributor

Imports resource directly in cliquet and change BaseResource to Resource. As per issue #34

@hiromipaw
Copy link
Contributor Author

I have some dependency to fix. Sorry for being late with this.

@hiromipaw hiromipaw changed the title Refactor the imports Refactor the imports [WIP] May 25, 2015
from cliquet import schema
from cliquet import utils


class BookmarkSchema(resource.ResourceSchema):
class BookmarkSchema(cliquet.ResourceSchema):
Copy link
Contributor

Choose a reason for hiding this comment

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

I suppose we could even rename cliquet.ResourceSchema to cliquet.Schema ?

Copy link
Contributor

Choose a reason for hiding this comment

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

+1

Copy link
Contributor Author

Choose a reason for hiding this comment

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

:) allrighty

@leplatrem
Copy link
Contributor

Please don't say you're late :)
We're still breaking things anyway :)

Thanks a lot for your contribs!!

@hiromipaw
Copy link
Contributor Author

Hey I have the same tests failing also on master. Is something with my local setup or am I forgetting something basic... ?

@@ -1,5 +1,5 @@
import mock

import cliquet
Copy link
Contributor

Choose a reason for hiding this comment

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

Is it necessary?

@Natim Natim changed the title Refactor the imports [WIP] Refactor the imports - Refs #34 May 29, 2015
@Natim
Copy link
Contributor

Natim commented May 29, 2015

refs nopressurelabs#1

@hiromipaw
Copy link
Contributor Author

Thanks for the comments. I'll keep working on this. :)

@hiromipaw hiromipaw closed this Jul 20, 2015
glasserc pushed a commit that referenced this pull request May 20, 2016
Default bucket as built-in plugin (fixes #277, fixes #311)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants