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

remove deprecated is_generic_easyblock from easyconfig.py #4243

Merged
merged 1 commit into from Apr 15, 2023

Conversation

branfosj
Copy link
Member

No description provided.

@branfosj branfosj added this to the 5.0 milestone Apr 15, 2023
@branfosj branfosj mentioned this pull request Apr 15, 2023
8 tasks
@boegel boegel added the change label Apr 15, 2023
@@ -3751,22 +3751,6 @@ def test_get_paths_for(self):
if env_eb_script_path:
os.environ['EB_SCRIPT_PATH'] = env_eb_script_path

def test_is_generic_easyblock(self):
Copy link
Member

Choose a reason for hiding this comment

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

also being tested via test/framework/filetools.py, so indeed OK to remove 👍

Copy link
Member

@boegel boegel left a comment

Choose a reason for hiding this comment

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

lgtm

@boegel boegel enabled auto-merge April 15, 2023 09:54
@boegel boegel merged commit cdf745f into easybuilders:5.0.x Apr 15, 2023
32 checks passed
@branfosj branfosj deleted the generic branch April 15, 2023 13:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants