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

[WIP] Clean Code inside the pipelines #1634

Closed
wants to merge 68 commits into from
Closed

Conversation

syeleti-msft
Copy link
Member

@syeleti-msft syeleti-msft commented Feb 18, 2025

This PR is currently work in progress and suggestions are always welcome.

Type of Change

  • Bug fix
  • New feature
  • Code quality improvement
  • Other (describe):

Description

Current pipeline code is not so idiomatic.

  • Feature / Bug Fix: (Brief description of the feature or issue being addressed)
    This PR tries to clean up the pipeline code by using same convention of writing yaml across all the pipelines and also encourages to use the templates whereever it is necessary.
    Changes are currently done for the CI, nightly pipeline to limit the amount of change. Soon all the pipelines would get uniform interface.

How Has This Been Tested?

Checklist

  • The purpose of this PR is explained in this or a referenced issue.
  • Tests are included and/or updated for code changes.
  • Documentation update required.
  • Updates to module CHANGELOG.md are included.
  • License headers are included in each file.

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