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

Add configs #7

Merged
merged 8 commits into from
Jul 12, 2024
Merged

Add configs #7

merged 8 commits into from
Jul 12, 2024

Conversation

ybiquitous
Copy link
Member

@ybiquitous ybiquitous enabled auto-merge (squash) July 12, 2024 06:08
@ybiquitous ybiquitous merged commit ad0bbf4 into main Jul 12, 2024
222 checks passed
@ybiquitous ybiquitous deleted the add-configs branch July 12, 2024 06:26
@Mouvedia
Copy link
Member

Some of the tests show up as passed but have failed.
It's probably not related to Stylelint though.
e.g. https://github.com/ybiquitous/stylelint-ecosystem-tester/actions/runs/9903277646/job/27358541572 is probably missing --experimental-vm-modules

@ybiquitous
Copy link
Member Author

Right. I believe most errors are unrelated to Stylelint: failed repositories' problems.

We can suggest a fix in such repositories or just remove them from this test suite. In my personal opinion, the former is preferred, though.

@Mouvedia
Copy link
Member

I believe most errors are unrelated to Stylelint: failed repositories' problems.

👍

Some are even failing at the install step though.
e.g. https://github.com/ybiquitous/stylelint-ecosystem-tester/actions/runs/9903277646/job/27358550823
Anyhow, this list of packages is a good starting point but we need to be careful about the passing state, it's not reliable.

@ybiquitous
Copy link
Member Author

We can see all errors and warnings in the annotation list in one page.
E.g. https://github.com/ybiquitous/stylelint-ecosystem-tester/actions/runs/9903754245

image

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