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 trailing whitespaces from .lud files #5

Closed
wants to merge 1 commit into from
Closed

Remove trailing whitespaces from .lud files #5

wants to merge 1 commit into from

Conversation

fbarbe00
Copy link

@fbarbe00 fbarbe00 commented Dec 7, 2021

Purpose

Remove 143437 white spaces from ludii game files (human linter :) ) in dev

Specifications

Use simple python script:

for ludii_file in ludii_files:
    lud = open(ludii_file, 'r').read()
    n_lud = '\n'.join(line.rstrip() for line in lud.splitlines())
    # append a newline to the end of the file
    n_lud += '\n'
    open(ludii_file, 'w').write(n_lud)

Purpose
=======
Remove 143445 white spaces from ludii game files (human linter :) )

Specifications
=============
Use simple python script:
```py
for ludii_file in ludii_files:
    lud = open(ludii_file, 'r').read()
    n_lud = '\n'.join(line.rstrip() for line in lud.splitlines())
    # append a newline to the end of the file
    n_lud += '\n'
    open(ludii_file, 'w').write(n_lud)
```
@DennisSoemers
Copy link
Collaborator

Thanks. It turns out that Clobber no longer compiles with these changes. Apparently, options break if there is no whitespace after an (item line. I don't think this is intended though, probably we should look at fixing it on our end, but I'll hold off on the merge for now.

@fbarbe00 fbarbe00 closed this by deleting the head repository Mar 16, 2023
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.

None yet

2 participants