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

API-44751 Testing environment contents #232

Open
wants to merge 48 commits into
base: main
Choose a base branch
from
Open

API-44751 Testing environment contents #232

wants to merge 48 commits into from

Conversation

SloopKoning
Copy link
Contributor

Testing contents of environment variables and embedded actions
in order to understand their content.

Copy link

These changes have been pushed to dev.

Copy link

These changes have been pushed to dev.

Copy link

These changes have been pushed to dev.

Copy link

These changes have been pushed to dev.

Copy link

These changes have been pushed to dev.

Copy link

These changes have been pushed to dev.

Copy link

These changes have been pushed to dev.

Copy link

These changes have been pushed to dev.

Copy link

These changes have been pushed to dev.

This commit introduces a new shell script that analyzes the unique
changes in the current Git branch compared to the base branch
(main or master). It captures added, modified, renamed, and deleted
files for each commit in the branch. This tool helps in tracking
the evolution of the branch and facilitates better management of
code changes.

The script includes debugging utilities to assist developers in
monitoring the process. It also handles scenarios like detached
HEAD states and non-existent base branches, improving user
experience and guiding them on necessary actions.
Copy link

These changes have been pushed to dev.

The checkout logic has been modified to always use the main branch
and the diff calculation now writes the result directly to a file.
This allows for easier usage in subsequent steps of the workflow
and improves performance by avoiding the need to output the diff
as a step variable.

Changes have also been made to the changed_files.sh script, which
now assumes that the base branch is main when running locally, and
simplifies the logic for classifying the changed and deleted files.
…ts execuatable premission set which apparently is a best practice in GH actions
Copy link

These changes have been pushed to dev.

The get_raw_git_diff.yaml workflow has been removed as it is no longer needed and was only used
during testing.

all changes were made to optimize performance in workflows and correcting bugs where files with spaces were not being correctly processed.  Currently we have no file paths were spaces or special
characters are used; however, if a future changed were to add these kinds of filepaths
then the existing code would break.

Updated the way that DATE_STRING was being set to ensure consistent code in all steps.
The new way of setting the DATE_STRING value is more readable / maintainable.
Copy link

These changes have been pushed to dev.

@SloopKoning SloopKoning marked this pull request as ready for review February 21, 2025 19:54
@SloopKoning SloopKoning requested review from a team as code owners February 21, 2025 19:54
@SloopKoning SloopKoning enabled auto-merge (squash) February 21, 2025 19:54
Copy link

These changes have been pushed to dev.

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