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

Support for Windows #250

Merged
merged 13 commits into from
Jan 9, 2023
Merged

Support for Windows #250

merged 13 commits into from
Jan 9, 2023

Conversation

mizdra
Copy link
Owner

@mizdra mizdra commented Jan 9, 2023

close: #217

@mizdra mizdra marked this pull request as ready for review January 9, 2023 14:08
# so it is difficult to run on windows. Therefore,
# we are temporarily skipping the E2E test on windows.
# TODO: Enable E2E test on windows.
if: ${{ matrix.os != 'windows-latest' }}
Copy link
Owner Author

Choose a reason for hiding this comment

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

ref: #251

Copy link
Owner Author

Choose a reason for hiding this comment

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

E2E tests could not be run, but I manually checked that it works. eslint-interactive now works on Windows too 🎉

However, there is a bug that the printing of problem details using the pager does not work on windows. I will fix this later.

@mizdra mizdra changed the title Fix windows Support for Windows Jan 9, 2023
mizdra and others added 3 commits January 9, 2023 23:17
Co-authored-by: Daniel Bartholomae <daniel@bartholomae.name>
Co-authored-by: Daniel Bartholomae <daniel@bartholomae.name>
@mizdra mizdra added the Type: Bug Bug or Bug fixes label Jan 9, 2023
@mizdra mizdra merged commit 6c86cc3 into main Jan 9, 2023
@mizdra mizdra deleted the fix-windows branch January 9, 2023 14:52
@mizdra mizdra added Type: Feature New Feature and removed Type: Bug Bug or Bug fixes labels Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature New Feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

command failed in Windows system
1 participant