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

Add molecule 4 compatibility #19

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

anxstj
Copy link

@anxstj anxstj commented Jul 12, 2022

I believe that this fixes #18.

@anxstj
Copy link
Author

anxstj commented Aug 9, 2022

@abtreece it seems that the pipeline is broken.

Do you have any objections?

@abtreece
Copy link
Collaborator

@anxstj could you squash your commits and repush? I re-enabled the Tox pipeline, so it should run tests

Move _get_tests() call to execute() that has action_args
@anxstj
Copy link
Author

anxstj commented Sep 29, 2022

@abtreece Thanks for taking a look at this.
I squashed and repushed as suggested, but the pipeline is still failing with python@python3 exceeds tier max size 250MiB: 329.4MiB.

@abtreece
Copy link
Collaborator

OK, I'm probably going to need to do some work to update the workflows to get them back to functional. I'll try to make some progress on it in the next week or so.

@disaster37
Copy link

Any news ? Some problem here ;)

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.

TypeError: Goss.execute() takes 1 positional argument but 2 were given
3 participants