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

CI: Updated parser doc in scripts/d.wms.py to fix E501 #4356

Closed
wants to merge 2 commits into from

Conversation

arohanajit
Copy link
Contributor

Fixed flake8: E501 in scripts/d.wms.py. I just split description into multiple lines to comply with max-line-length

@github-actions github-actions bot added GUI wxGUI related Python Related code is in Python labels Sep 21, 2024
@neteler neteler changed the title CI: Updated doc in scripts/ to fix E501 CI: Updated parser doc in scripts/d.wms.py to fix E501 Sep 21, 2024
@neteler
Copy link
Member

neteler commented Sep 21, 2024

Does the parser accept multiple description lines?

@arohanajit
Copy link
Contributor Author

If I am being honest I don't really know. I was hoping to rely on github actions to highlight any potential errors. If you could tell me how can I check for this (I am new to this repo) that would be really helpful! Considering how common this error is

@wenzeslaus
Copy link
Member

wenzeslaus commented Sep 21, 2024 via email

@arohanajit arohanajit closed this Sep 21, 2024
@neteler
Copy link
Member

neteler commented Sep 22, 2024

I was hoping to rely on github actions to highlight any potential errors.

(In this case there is AFAIK no test implemented, hence the github actions can not catch any potential error.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
GUI wxGUI related Python Related code is in Python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants