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

Cura 10831 #17475

Merged
merged 47 commits into from
Dec 8, 2023
Merged

Cura 10831 #17475

merged 47 commits into from
Dec 8, 2023

Conversation

jellespijker
Copy link
Member

@jellespijker jellespijker commented Nov 29, 2023

Description

Ported a lot of workflow to use the reusable workflows in the new Cura-workflows repository. See PR: Ultimaker/cura-workflows#1

The biggest impact on this repository is that the version of the conan package is now determined by the version key in conandata.yml and no longer determined based on the git tag. During a push the Conan user, channel is determined based Github runner context, see the conan-recipe-versions.yml in the Cura-workflows and the semver build metadata is added from the sha of the reference name (git hash).

Furthermore the Cura dependencies maintained by UltiMaker (e.q.: fdm_materials, uranium, curaengine) are no longer specified in the conanfile.py but are also present in the conandata.yml which allows us to automate the user and channel on main and release branches in the future. Since it is a yaml file which can be read in and easily changed by an automated workflow. For instance change the Conan channel to stable when we create the release branch. Or pin to specific conan package release version (e.q.: curaengine/5.7.0@_/_) when we create an actual release.

When you now create a release branch now you would need to do this manually.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Printer definition file(s)
  • Translations

How Has This Been Tested?

  • Locally
  • Remote

Test Configuration:

  • Operating System: Linux, Windows, Mac

Checklist:

This will allow workflow automations to easily create release branches and set the versions

Contributes to CURA-10831
This commit updates the Uranium dependency in the Conan file and introduces an internationalization (i18n) option.
It refactor the devtool checks to check for the "enable_i18n" option instead.
Lastly, cleanups and adjustments are done in preparation for enabling i18n in the build process.

Contributes to CURA-10831
should be `enable_i18n`

Contributes to CURA-10831
Removed redundant workflow dispatch inputs and comments in .github/workflows/conan-package.yml. These were unnecessary
for the push-based workflow and have been simplified to streamline the process and make it more efficient.

Contributes to CURA-10831
# Conflicts:
#	conanfile.py
# Conflicts:
#	.github/workflows/conan-package-create.yml
#	.github/workflows/conan-package.yml
conanfile.py Outdated
Comment on lines 326 to 327
if "fdm_materials" in req:
continue
Copy link
Contributor

Choose a reason for hiding this comment

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

Why would fdm_materials be in the internal conan requirements?

Copy link
Member Author

Choose a reason for hiding this comment

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

Otherwise you will endup with a conflict when building with the internal option True

@jellespijker jellespijker marked this pull request as ready for review December 5, 2023 11:10
@jellespijker jellespijker mentioned this pull request Dec 6, 2023
10 tasks
The attribute 'enterprise' in conanfile.py was incorrectly named and has been renamed to 'internal'.
This revision ensures that options are accessed correctly to avoid potential issues in usage.

Contributes to CURA-10831
.github/workflows/conan-package.yml Outdated Show resolved Hide resolved
.github/workflows/conan-package.yml Outdated Show resolved Hide resolved
.github/workflows/installers.yml Outdated Show resolved Hide resolved
.github/workflows/installers.yml Outdated Show resolved Hide resolved
.github/workflows/installers.yml Outdated Show resolved Hide resolved
.github/workflows/windows.yml Outdated Show resolved Hide resolved
.github/workflows/windows.yml Outdated Show resolved Hide resolved
conandata.yml Outdated Show resolved Hide resolved
.github/workflows/installers.yml Outdated Show resolved Hide resolved
.github/workflows/conan-package.yml Outdated Show resolved Hide resolved
Co-authored-by: Casper Lamboo <c.lamboo@ultimaker.com>
@jellespijker jellespijker merged commit 47f3e01 into main Dec 8, 2023
4 of 6 checks passed
@jellespijker jellespijker deleted the CURA-10831 branch December 8, 2023 13:36
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