Skip to content

Added Version control github action#3

Merged
arghyabi merged 1 commit into
masterfrom
version_check_add
Sep 16, 2025
Merged

Added Version control github action#3
arghyabi merged 1 commit into
masterfrom
version_check_add

Conversation

@arghyabi
Copy link
Copy Markdown
Owner

No description provided.

@arghyabi arghyabi requested a review from Copilot September 16, 2025 08:29
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a GitHub Actions workflow to enforce version control requirements by ensuring that config.yaml and its appVersion field are updated in pull requests.

  • Adds automated validation for version file updates
  • Enforces that both the config file is modified and the appVersion field is changed
  • Configured to run on pull request events (opened and synchronize)

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.


- name: Check if appVersion in config.yaml has been updated
run: |
VERSION_CHANGE=$(git diff HEAD^ HEAD config.yaml | grep 'appVersion')
Copy link

Copilot AI Sep 16, 2025

Choose a reason for hiding this comment

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

The grep command will match any line containing 'appVersion', including comments or documentation. This could lead to false positives. Consider using a more specific pattern like grep '^[[:space:]]*appVersion:' to match only the actual configuration field.

Suggested change
VERSION_CHANGE=$(git diff HEAD^ HEAD config.yaml | grep 'appVersion')
VERSION_CHANGE=$(git diff HEAD^ HEAD config.yaml | grep '^[[:space:]]*appVersion:')

Copilot uses AI. Check for mistakes.
Signed-off-by: Arghya Biswas <arghyabiswas05@gmail.com>
@arghyabi arghyabi merged commit 29dee42 into master Sep 16, 2025
1 check passed
@arghyabi arghyabi deleted the version_check_add branch September 16, 2025 08:31
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.

2 participants