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 10.2.0 #165

Merged
merged 7 commits into from
Feb 23, 2024
Merged

Release 10.2.0 #165

merged 7 commits into from
Feb 23, 2024

Conversation

folmos-at-orange
Copy link
Member

No description provided.

popescu-v and others added 7 commits February 19, 2024 14:37
…nch-on-ubuntu

Fix JAVA_HOME setting in Debian post-install script
* Use the right communicator
"max error flow" messages were not received and remained in the communicator

* code simplification
We use the working slave list instead of the content of the task communicator.
It is the right way because we send messages in the world comm and not in the task comm.

* Add BugMPIWithErrors in LearningTest/Standard
This allows to cancel any ongoing run of the workflow if it is in the
same PR or git reference. This is to enable faster development cycles in
the CI by not having to cancel runs manually.
We add a release job that is only executed when the workflow executed on
a tag (either by pushing or by manual execution). The workflow creates
the release as a pre-release so it does not disturb the current stable
one in the github page. The workflow may be executed many times as long
as the release stays in the "pre-release" state.

In the release step, we rename the package file names so their version
match the tag. For now, the maintainers must check the coherence of the
"source version" (that in KWKhiopsVersion.cpp) when creating a release.

The release step depends only ond the build step and the tests steps
are now allowed to fail. It is to maintainers to clean any defectiva
package from a final release.

Other opportunistic changes in this commit:
- Update to version 4 of actions checkout, upload-artifact and
  download-artifact. This is to avoid deprecation warnings and in the
  case of dowload-artifact it even necessary as we use the new "pattern"
  option.
- Simplify names of workflows and jobs so they are better seen in the
  "checks" box and tab of a PR. The verbose steps names are kept.  This
  is to achieve a good compromise between legibility in the GitHub webUI
  and auto-documentation.
…upload-to-a-release

Add release to package build CICD
Copy link
Contributor

@bruno-at-orange bruno-at-orange left a comment

Choose a reason for hiding this comment

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

LGTM

@folmos-at-orange folmos-at-orange merged commit f4b1494 into main Feb 23, 2024
74 of 78 checks passed
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