Skip to content

Move CI config from repo root #183

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

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

Conversation

Alexsandruss
Copy link
Contributor

@Alexsandruss Alexsandruss commented Apr 25, 2025

Description


PR should start as a draft, then move to ready for review state after CI is passed and all applicable checkboxes are closed.
This approach ensures that reviewers don't spend extra time asking for regular requirements.

You can remove a checkbox as not applicable only if it doesn't relate to this PR in any way.

Checklist to comply with before moving PR from draft:

PR completeness and readability

  • I have reviewed my changes thoroughly before submitting this pull request.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have updated the documentation to reflect the changes or created a separate PR with update and provided its number in the description, if necessary.
  • Git commit message contains an appropriate signed-off-by string (see CONTRIBUTING.md for details).
  • I have added a respective label(s) to PR if I have a permission for that.
  • I have resolved any merge conflicts that might occur with the base branch.

Testing

  • I have run it locally and tested the changes extensively.
  • All CI jobs are green or I have provided justification why they aren't.
  • I have extended testing suite if new functionality was introduced in this PR.

@Alexsandruss
Copy link
Contributor Author

/azp run CI

Copy link

Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@Alexsandruss
Copy link
Contributor Author

/azp run CI

Copy link

Azure Pipelines failed to run 1 pipeline(s).

@Alexsandruss Alexsandruss added the enhancement New feature or request label Apr 25, 2025
@Alexsandruss Alexsandruss marked this pull request as ready for review April 25, 2025 12:24
@Alexsandruss
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines failed to run 1 pipeline(s).

@Alexsandruss
Copy link
Contributor Author

/azp run

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@Alexsandruss Alexsandruss force-pushed the dev/ci-conf-dir-update branch from d0e09a9 to b3811df Compare April 25, 2025 12:38
@Alexsandruss Alexsandruss marked this pull request as draft April 25, 2025 12:56
@Alexsandruss
Copy link
Contributor Author

Need to change azupe pipeline file location on https://dev.azure.com/daal/scikit-learn_bench/_build?definitionId=21

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant