-
-
Notifications
You must be signed in to change notification settings - Fork 373
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
Integrate code improvements and add linters #307
Commits on Apr 27, 2023
-
🔧 chore(linters): add linters for YAML, Ansible, and Python files
This commit adds linters for YAML, Ansible, and Python files. The `.yamllint` file is updated to ignore the `.venv` directory and to enforce a minimum of one space between comments and content. The `.ansible-lint` file is updated to exclude the `.venv` directory. The `Makefile` is updated to include the new `linters.mak` file, which contains the targets for the linters. The linters are executed in the `lint` target.
Configuration menu - View commit details
-
Copy full SHA for 860209c - Browse repository at this point
Copy the full SHA 860209cView commit details -
🔧 chore(molecule.yml): remove commented out code and fix indentation
This commit removes commented out code and fixes indentation in the molecule.yml file. The commented out code is not needed and the indentation is inconsistent, which can make the file harder to read.
Configuration menu - View commit details
-
Copy full SHA for 53700aa - Browse repository at this point
Copy the full SHA 53700aaView commit details -
🚀 chore(.gitpod.Dockerfile): add bash-completion and remove redundant…
… packages The bash-completion package is added to enable tab completion in the terminal. The following packages are removed as they are either redundant or not required: sudo, nano, iproute2, curl, htop, gnupg, and lsb-release.
Configuration menu - View commit details
-
Copy full SHA for 6ea9a94 - Browse repository at this point
Copy the full SHA 6ea9a94View commit details -
🎨 style(linters.mak): add lint target to Makefile
The lint target is added to the Makefile to run all linters at once. This improves the development workflow by allowing developers to run all linters with a single command.
Configuration menu - View commit details
-
Copy full SHA for a4f622c - Browse repository at this point
Copy the full SHA a4f622cView commit details -
🆕 chore(.flake8): add flake8 configuration file
A new flake8 configuration file has been added to the .config directory. This file specifies the directories and files to exclude from flake8 linting.
Configuration menu - View commit details
-
Copy full SHA for 04d7616 - Browse repository at this point
Copy the full SHA 04d7616View commit details -
🚚 chore(linters.mak): move flake8 configuration file to .config direc…
…tory The flake8 configuration file was moved from the root directory to the .config directory to improve organization and avoid cluttering the root directory.
Configuration menu - View commit details
-
Copy full SHA for 94be5dc - Browse repository at this point
Copy the full SHA 94be5dcView commit details -
🔧 chore(linters.mak): remove comments and echo statements
🔧 chore(postgrespro): move vars file to vars directory The comments and echo statements were removed from the linters.mak file to reduce noise in the output. The vars file was moved to the vars directory to improve organization and consistency with other roles.
Configuration menu - View commit details
-
Copy full SHA for a4af2a0 - Browse repository at this point
Copy the full SHA a4af2a0View commit details -
📝 docs(CONTRIBUTING.md): update contributing guidelines
This commit updates the contributing guidelines to include instructions on how to lint the codebase using `make lint`. It also adds a step to the existing instructions to test with linters.
Configuration menu - View commit details
-
Copy full SHA for 52b1cd8 - Browse repository at this point
Copy the full SHA 52b1cd8View commit details
Commits on Apr 28, 2023
-
🔀 chore(converge.yml): remove redundant ansible.builtin prefix from i…
…nclude_vars module The ansible.builtin prefix is redundant and can be removed from the include_vars module.
Configuration menu - View commit details
-
Copy full SHA for c883c8a - Browse repository at this point
Copy the full SHA c883c8aView commit details -
🔥 chore(molecule.mak): remove molecule-lint target
The molecule-lint target has been removed as it is no longer needed.
Configuration menu - View commit details
-
Copy full SHA for c0e72bf - Browse repository at this point
Copy the full SHA c0e72bfView commit details