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

nixos/{lib,maintainers,tests}: editorconfig fixes #94607

Merged
merged 3 commits into from Aug 3, 2020
Merged

nixos/{lib,maintainers,tests}: editorconfig fixes #94607

merged 3 commits into from Aug 3, 2020

Conversation

zowoq
Copy link
Contributor

@zowoq zowoq commented Aug 3, 2020

No description provided.

@zowoq zowoq merged commit 8ae7f8c into NixOS:master Aug 3, 2020
@zowoq zowoq deleted the ecc-nixos branch August 3, 2020 14:24
@Ma27
Copy link
Member

Ma27 commented Aug 3, 2020

Did you actually test this? The systemd-networkd-vrf test is now broken for instance.

@Ma27
Copy link
Member

Ma27 commented Aug 3, 2020

Fixed in 5bd1fb2.

@zowoq
Copy link
Contributor Author

zowoq commented Aug 3, 2020

Add to the .editorconfig. 1ccda2f

I'd suggest adding a comment to the test as well as some editors may ignore the config.

Ma27 added a commit that referenced this pull request Aug 3, 2020
…ces in test script

For further context please read the discussion in #94607 (comment)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants