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

Release v2.2.1.1 #105

Merged
merged 14 commits into from
May 27, 2021
Merged

Release v2.2.1.1 #105

merged 14 commits into from
May 27, 2021

Conversation

IshaanDesai
Copy link
Member

@IshaanDesai IshaanDesai commented May 18, 2021

This release is required due to a bug fix release of preCICE v2.2.1

Features of this release:

  • Removing Travis CI
  • Adding tests to check autopep8 formatting, building dockerimage

@IshaanDesai IshaanDesai self-assigned this May 18, 2021
@IshaanDesai
Copy link
Member Author

@BenjaminRodenberg is the Spack build expected to fail right now?

Copy link
Member

@BenjaminRodenberg BenjaminRodenberg left a comment

Choose a reason for hiding this comment

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

From my side this PR looks good.

The failing spack test is a tricky issue. I assume that this is a problem on precice@develop and not on [email protected]. However, we currently don't know, since we do not test using [email protected]. From my perspective a manual test with [email protected] should be sufficient for this release. On the long run we should think about our testing strategy

  1. for the python bindings, when the spack environment is updated (weekly cron job)
  2. for precice, since we currently don't have any information on the status of the spack package precice@develop.

@ajaust
Copy link
Collaborator

ajaust commented May 19, 2021

The failing Spack build might be simply a Spack artifact when using the Python module in a fresh Spack environment. However, the Spack environment is not touched/changed.

It could also be related #86 which means that the Python module is installed to a location where we do not expect/want it to be. Maybe the module is in the environment of a/the wrong Python installation. At least the output suggest that the Python modules was built and installed successfully.

@ajaust
Copy link
Collaborator

ajaust commented May 27, 2021

I just restarted the failing test twice and now it passed twice. I am not sure what to conclude here. I would guess this PR is fine for now.

@IshaanDesai IshaanDesai merged commit 2039557 into master May 27, 2021
@BenjaminRodenberg BenjaminRodenberg deleted the python-bindings-v2.2.1.1 branch March 21, 2024 17:02
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.

3 participants