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

feat: extract info on calculations/workflows #210

Merged
merged 4 commits into from
Mar 18, 2022

Conversation

ltalirz
Copy link
Member

@ltalirz ltalirz commented Feb 20, 2022

For plugins that can be installed successfully, inspect the process
classes provided by them (similar to what verdi plugin does) and store
this information in a local json file.

This is in preparation of a more detailed information shown on
the registry.

Also:

  • Switch installation tests to aiida-core docker container in order to
    get a clean install every time.

For plugins that can be installed successfully, inspect the process
classes provided by them (similar to what `verdi plugin` does) and store
this information in a local json file.

This is in preparation of a more detailed information shown on
the registry.

Also:
 - Switch installation tests to aiida-core docker container in order to
   get a clean install every time.
@ltalirz
Copy link
Member Author

ltalirz commented Feb 20, 2022

It's not clear to me

  • why this job can be "skipped"
  • why no output is being produced
  • what is causing the issues the second time we try to create a container (from here it looked like the first time we create the container everything runs fine).

@ltalirz ltalirz marked this pull request as ready for review March 18, 2022 11:14
@ltalirz
Copy link
Member Author

ltalirz commented Mar 18, 2022

I'm going to merge this, even if the github action still fails.

If it turns out it can't be made to work, we'll need to figure out something else.

@ltalirz ltalirz merged commit 199030b into aiidateam:master Mar 18, 2022
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.

1 participant