Skip to content

A0-1803: Switch to self-hosted runners #903

Merged
Marcin-Radecki merged 49 commits intomainfrom
A0-1803-switch-to-self-hosted-runners
Mar 2, 2023
Merged

A0-1803: Switch to self-hosted runners #903
Marcin-Radecki merged 49 commits intomainfrom
A0-1803-switch-to-self-hosted-runners

Conversation

@ghost
Copy link

@ghost ghost commented Feb 3, 2023

No description provided.

@ghost ghost self-assigned this Feb 3, 2023
@ghost ghost marked this pull request as draft February 3, 2023 23:12
@ghost ghost changed the title A0-1803: Switch to self-hoster runners [do not merge] A0-1803: Switch to self-hosted runners [do not merge] Feb 5, 2023
@Marcin-Radecki Marcin-Radecki marked this pull request as ready for review February 23, 2023 09:01
@Marcin-Radecki Marcin-Radecki changed the title A0-1803: Switch to self-hosted runners [do not merge] A0-1803: Switch to self-hosted runners Feb 23, 2023
@fbielejec
Copy link
Contributor

fbielejec commented Feb 27, 2023

This PR could be a great chance to clean up all the accumulated inconsistencies in the workflows. Just a few examples:

additionaly a lot of pipelines and literally days of developer time could be saved by adding paths argument to every workflow when it should run

Copy link
Contributor

@maciejnems maciejnems left a comment

Choose a reason for hiding this comment

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

Python tests LGTM

Copy link
Contributor

@obrok obrok left a comment

Choose a reason for hiding this comment

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

Onwards to a brighter CI future! 🚀

Copy link
Contributor

@mike1729 mike1729 left a comment

Choose a reason for hiding this comment

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

Wonderful 🎉

@Marcin-Radecki Marcin-Radecki merged commit cfaeccf into main Mar 2, 2023
@Marcin-Radecki Marcin-Radecki deleted the A0-1803-switch-to-self-hosted-runners branch March 2, 2023 10:55
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.

7 participants