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

Loki-transform: Remove custom entry point options #465

Draft
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

mlange05
Copy link
Collaborator

Note: This sits on top of PR #464 and requires CLOUDSC CLOUDSC PR 106 to be merged first.

Note: For testing only at this stage.

Following the conversion to config-file based pipelines, this PR now removes all the custom entry point options under loki-transform.py convert and their associated use in CMake macros. It also removes the now unused loki_transform_convert entry point.

@mlange05 mlange05 requested a review from reuterbal December 24, 2024 05:09
@mlange05 mlange05 changed the title Naml remove custom entry points Loki-transform: Remove custom entry point options Dec 24, 2024
Copy link

Documentation for this branch can be viewed at https://sites.ecmwf.int/docs/loki/465/index.html

@mlange05 mlange05 force-pushed the naml-remove-custom-entry-points branch from 63abe8d to bc4ac13 Compare January 9, 2025 15:59
@mlange05 mlange05 force-pushed the naml-remove-custom-entry-points branch from bc4ac13 to 2f21edd Compare January 9, 2025 18:24
Copy link

codecov bot commented Jan 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 93.35%. Comparing base (b0a2344) to head (2f21edd).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #465   +/-   ##
=======================================
  Coverage   93.35%   93.35%           
=======================================
  Files         223      223           
  Lines       41428    41428           
=======================================
  Hits        38676    38676           
  Misses       2752     2752           
Flag Coverage Δ
lint_rules 96.39% <ø> (ø)
loki 93.31% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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