Skip to content

Chore/prepare pgtools to feed analyzer#7

Merged
gmartinez-dbai merged 12 commits intomainfrom
chore/prepare-pgtools-to-feed-analyzer
Dec 9, 2025
Merged

Chore/prepare pgtools to feed analyzer#7
gmartinez-dbai merged 12 commits intomainfrom
chore/prepare-pgtools-to-feed-analyzer

Conversation

@gmartinez-dbai
Copy link
Copy Markdown
Collaborator

This pull request introduces a new CI workflow and improves documentation and code quality for the pgtools automation scripts. The main changes include adding a GitHub Actions workflow for automated testing and validation, updating documentation to reflect new verification commands and recommended practices, and refactoring shell scripts for better maintainability and compatibility.

CI/CD and Automation Enhancements

  • Added a GitHub Actions workflow (.github/workflows/ci.yml) to automate dependency installation, PostgreSQL service setup, and shell script validation with ShellCheck. This lays the foundation for future automated testing of the full suite.

Documentation Updates

  • Updated README.md, CONTRIBUTING.md, and automation/README.md to include instructions for running the new automation checks, HOT report verification, and the local pre-commit bundle. These changes clarify recommended development and verification steps. [1] [2] [3] [4] [5]

Shell Script Refactoring and Quality

  • Improved shell script compatibility and maintainability by refactoring variable assignments to avoid command substitution in local declarations and adding ShellCheck annotations to suppress false positives, especially for sourced configuration files and associative arrays. [1] [2] [3] [4] [5] [6] [7] [8] [9] [10] [11] [12] [13] [14] [15] [16] [17]

Minor Cleanups

  • Removed unused variable assignments in automation/export_metrics.sh and automation/pgtools_scheduler.sh for improved clarity. [1] [2]

These updates collectively make it easier to develop, verify, and maintain pgtools automation scripts with improved CI integration and clearer developer guidance.

@gmartinez-dbai gmartinez-dbai merged commit eec9116 into main Dec 9, 2025
2 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.

1 participant