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

style: fix some simple style issues that seem to happen a lot #897

Merged
merged 1 commit into from
Sep 27, 2023

Conversation

braindigitalis
Copy link
Contributor

This fixes instances where if has no space between itself and its bracket, or has no braces around a single statement associated with it, or its else, or has braces on the next line. This goes against the code style guide. Also fixed a few for() i noticed along the way etc, but didnt go out of my way to find other stuff. We will fix that "if and when we find it". LOL programmer pun.

Code change checklist

  • I have ensured that all methods and functions are fully documented using doxygen style comments.
  • My code follows the coding style guide.
  • I tested that my change works before raising the PR.
  • I have ensured that I did not break any existing API calls.
  • I have not built my pull request using AI, a static analysis tool or similar without any human oversight.

…going through the code match code style, if () must always have braces even if its on the same line
@braindigitalis braindigitalis self-assigned this Sep 27, 2023
@netlify
Copy link

netlify bot commented Sep 27, 2023

Deploy Preview for dpp-dev ready!

Name Link
🔨 Latest commit 42e3329
🔍 Latest deploy log https://app.netlify.com/sites/dpp-dev/deploys/65145ee5a998d00008b0e712
😎 Deploy Preview https://deploy-preview-897--dpp-dev.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Sep 27, 2023
@braindigitalis braindigitalis merged commit 6940529 into dev Sep 27, 2023
46 of 48 checks passed
@braindigitalis braindigitalis deleted the fix_orphan_if branch September 27, 2023 17:01
Copy link
Contributor

@Jaskowicz1 Jaskowicz1 left a comment

Choose a reason for hiding this comment

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

All good :)

@Jaskowicz1
Copy link
Contributor

oh this already merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants