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

Change the format of the version tracking file #379

Merged
merged 2 commits into from
Dec 10, 2024

Conversation

mcdonnnj
Copy link
Member

@mcdonnnj mcdonnnj commented Oct 10, 2024

🗣 Description

This pull request removes the Python trappings from the version tracking file (src/version.txt). The version tracking file is changed to be a flat text file storing the version.

💭 Motivation and context

When writing #377 I found myself questioning why the version was stored as it currently is and decided it should be changed.

🧪 Testing

Automated tests pass. I confirmed that the bump_version.sh script worked as expected.

✅ Pre-approval checklist

  • This PR has an informative and human-readable title.
  • Changes are limited to a single goal - eschew scope creep!
  • All relevant type-of-change labels have been added.
  • I have read the CONTRIBUTING document.
  • These code changes follow cisagov code standards.
  • Tests have been added and/or modified to cover the changes in this PR.
  • All new and existing tests pass.

@mcdonnnj mcdonnnj added breaking change This issue or pull request involves changes to existing functionality improvement This issue or pull request will add or improve functionality, maintainability, or ease of use hacktoberfest-accepted Pull request that should count toward Hacktoberfest participation labels Oct 10, 2024
@mcdonnnj mcdonnnj self-assigned this Oct 10, 2024
@mcdonnnj mcdonnnj force-pushed the improvement/change_version_file_format branch from cf59ea7 to 13c5ef6 Compare October 10, 2024 20:55
Copy link
Member

@dav3r dav3r left a comment

Choose a reason for hiding this comment

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

👍

@mcdonnnj mcdonnnj added the kraken 🐙 This pull request is ready to merge during the next Lineage Kraken release label Oct 11, 2024
@jsf9k jsf9k added this pull request to the merge queue Dec 9, 2024
@jsf9k jsf9k removed this pull request from the merge queue due to a manual request Dec 9, 2024
github-merge-queue bot pushed a commit that referenced this pull request Dec 9, 2024
…format

Change the format of the version tracking file
There is no reason to have any information but the version in the
version tracking file for Packer template projects. The current format
is simply an artifact of mirroring things over from the
cisagov/skeleton-python-library project.
@jsf9k jsf9k force-pushed the improvement/change_version_file_format branch from 13c5ef6 to 4934005 Compare December 10, 2024 15:27
The src directory no longer exists.
@jsf9k jsf9k added this pull request to the merge queue Dec 10, 2024
Merged via the queue into develop with commit eb4508a Dec 10, 2024
11 checks passed
@jsf9k jsf9k deleted the improvement/change_version_file_format branch December 10, 2024 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking change This issue or pull request involves changes to existing functionality hacktoberfest-accepted Pull request that should count toward Hacktoberfest participation improvement This issue or pull request will add or improve functionality, maintainability, or ease of use kraken 🐙 This pull request is ready to merge during the next Lineage Kraken release
Projects
Development

Successfully merging this pull request may close these issues.

3 participants