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

REMOVE THE WINDOWS SLAVE FROM THE CI #16842

Closed
Ducasse opened this issue Jun 29, 2024 · 5 comments
Closed

REMOVE THE WINDOWS SLAVE FROM THE CI #16842

Ducasse opened this issue Jun 29, 2024 · 5 comments
Labels

Comments

@Ducasse
Copy link
Member

Ducasse commented Jun 29, 2024

We must disable the window slave.
It is useless to have something that does nothing and say that everything is working.
We should grow up.

Then why do we change all the time the labels of the issue.
To me this is a showstopper.
I cannot work when I have to remember bullshit.

@astares
Copy link
Member

astares commented Jun 30, 2024

Yes, lets ignore the most prominent OS used in business setups and keep pharo an experiment for apple only

@Rinzwind
Copy link
Contributor

Related: issue #15105 (“[CI] Windows CI run sometimes is green but did not run”).

@astares
Copy link
Member

astares commented Jun 30, 2024

Anything one can do from the outside to support on this issue? Is there any description of the CI setup so that it is possible to try to verify issues caused?

@Ducasse
Copy link
Member Author

Ducasse commented Jun 30, 2024

No.

Now keeping a CI that does not do anything and mention in permanence that everything is ok and that all the tests are ok is not a good practice.

  • The first step is remove the CI job
  • The second step is fix the CI and reenable it.

@guillep
Copy link
Member

guillep commented Jul 2, 2024

Fixed, solution is in #16845

@guillep guillep closed this as completed Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants