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 whitespace and quotes formatting #1086

Merged
merged 2 commits into from
Nov 21, 2023

Conversation

hyperupcall
Copy link
Contributor

Description

When using the ESLint VScode extension, many files will have red squigly lines, which represent formatting errors. This may be a source of confusion for people new to the codebase.

Additionally, the EditorConfig file is not configured to configure an indent_type space for files that require it or should use it, such as yaml and markdown.

This fixes both of those situations, fixing the inconsistent whitespace.

Related to #879 (f9d328c, e35768e, and a4e0244)

Type of change

  • Note merging this changes the database configuration.
  • This change requires a documentation update

Checklist

  • I have followed the OED pull request ideas
  • I have removed text in ( ) from the issue request
  • You acknowledge that every person contributing to this work has signed the OED Contributing License Agreement and each author is listed in the Description section.

Limitations

N/A

database: process.env.OED_DB_TEST_DATABASE
host: process.env.POSTGRES_HOST,
port: process.env.POSTGRES_PORT,
user: 'postgres',

Check failure

Code scanning / CodeQL

Hard-coded credentials Critical

The hard-coded value "postgres" is used as
user name
.
Copy link
Member

@huss huss left a comment

Choose a reason for hiding this comment

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

Thanks to @hyperupcall for another task completed. This has also annoyed me so I appreciate the change. Everything looks good.

For other developers: OED will do an npm install because of spacing changes in the package files. However, there are not actual changes to the packages.

@huss
Copy link
Member

huss commented Nov 21, 2023

Note the CodeQL error is not new and needs to be dealt without outside this pull request.

@huss huss merged commit fa05070 into OpenEnergyDashboard:development Nov 21, 2023
2 of 3 checks passed
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