search

UCL-COMP0233-2022-2023 / RSE-Classwork

11 stars 67 forks source link

💄🐍 code - linting #47

Open dpshelio opened 2 years ago

dpshelio commented 2 years ago

  1. Analyse the awesome Charlene's package you've created (#46) and run one of the linting tools You may need to install them. Which messages did you get? Was your IDE (e.g., VS Code) warning you of it already?

  2. Fix them either manually or automatically using a code formatter (e.g., yapf or black)

  3. Can you think of a way of checking the style is checked before merging new contributions? Add your suggestions below 👇.

  4. Share your solution even if it's a work-in-progress as a pull request to Charlene's repository mentioning this issue (by including the text Addresses UCL-COMP0233-22-23/RSE-Classwork#47 in the pull request description), remember to mention your team members too! (with @github_username)

bwan1 commented 2 years ago

You could include the CL linting command in the workflow yml file