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

fix to cli scaffold controller spec deprecated garnetspec module #1296

Merged
merged 1 commit into from
Sep 24, 2022

Conversation

rrevi
Copy link
Contributor

@rrevi rrevi commented Sep 22, 2022

Description of the Change

At some point in the history of the GarnetSpec shard, the GarnetSpec::Controller::Test module was deprecated, instead the GarnetSpec::SystemTest module is to be used.

Benefits

Fixes an error when running cli scaffold controller spec files in more current versions of the GarnetSpec shard.

Possible Drawbacks

Will not work in older versions of GarnetSpec shard.

Copy link
Member

@crimson-knight crimson-knight left a comment

Choose a reason for hiding this comment

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

Nice catch, looks like there are also 2 other places this needs to be updated. I can open a separate PR for that.

@crimson-knight crimson-knight merged commit 4b8dc3c into amberframework:master Sep 24, 2022
@rrevi rrevi deleted the fix-garnetspec-error branch September 28, 2022 01:22
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.

2 participants