Skip to content
This repository has been archived by the owner on Jan 17, 2021. It is now read-only.

Add tox support #10

Closed
wants to merge 5 commits into from
Closed

Add tox support #10

wants to merge 5 commits into from

Conversation

uilianries
Copy link
Member

@uilianries uilianries commented Dec 1, 2020

  • Use Tox to run tests
  • Add Conan develop for testing
  • Copy Conan test Utils
    Conan doesn't distribute test folder under python package, we have to copy if we want to use ClientTest/ServerTest. If we have to re-use it, I suggest packaging as a separated package.

Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
@Croydon
Copy link
Member

Croydon commented Dec 4, 2020

I don't know how to feel about this. I rather hope that this tool becomes obsolete through new/better features in Conan and Conan Package Tools

e.g.

conan-io/conan-package-tools#282

slightly related (maybe):
conan-io/conan#5453 (though this one is only local for now)

@Croydon
Copy link
Member

Croydon commented Dec 13, 2020

I have created bincrafters/community#1320

@Croydon Croydon changed the base branch from master to main December 13, 2020 18:40
@Croydon
Copy link
Member

Croydon commented Dec 14, 2020

If we move this forward nonetheless, it requires now a rebase on the main branch since I moved the CI to GHA, sorry!

I have merged two small changes from this PR: ad351f3 & 4822742

@Croydon
Copy link
Member

Croydon commented Jan 17, 2021

This tool is now obsolete. See bincrafters/community#1320

@Croydon Croydon closed this Jan 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants