Skip to content

Deprecate the YAML pipeline syntax #2904

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

Merged
merged 7 commits into from
Feb 2, 2023

Conversation

dominiklohmann
Copy link
Member

This deprecates the old YAML pipeline syntax, and makes it possible to use the new string syntax also for pipelines defined in the config file.

@dominiklohmann dominiklohmann marked this pull request as ready for review February 2, 2023 10:37
@dominiklohmann dominiklohmann force-pushed the topic/deprecate-yaml-pipeline-syntax branch from 86f678f to 21e7736 Compare February 2, 2023 10:38
@dominiklohmann dominiklohmann force-pushed the topic/deprecate-yaml-pipeline-syntax branch from 21e7736 to dc15179 Compare February 2, 2023 10:42
@dominiklohmann dominiklohmann force-pushed the topic/deprecate-yaml-pipeline-syntax branch from dc15179 to 9e36092 Compare February 2, 2023 10:44
@dominiklohmann dominiklohmann force-pushed the topic/deprecate-yaml-pipeline-syntax branch from 9e36092 to b6091a3 Compare February 2, 2023 11:47
Copy link

@Dakostu Dakostu left a comment

Choose a reason for hiding this comment

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

It's a baseline approval from me. I'm glad to see the new pipeline YAML syntax. I did leave some comments though.

@dominiklohmann dominiklohmann merged commit 076ef46 into master Feb 2, 2023
@dominiklohmann dominiklohmann deleted the topic/deprecate-yaml-pipeline-syntax branch February 2, 2023 19:23
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