Skip to content

workflow adjustments#665

Open
sfc-gh-jcieslak wants to merge 5 commits intomainfrom
jcieslak/workflow-adjustments
Open

workflow adjustments#665
sfc-gh-jcieslak wants to merge 5 commits intomainfrom
jcieslak/workflow-adjustments

Conversation

@sfc-gh-jcieslak
Copy link
Member

@sfc-gh-jcieslak sfc-gh-jcieslak commented Mar 11, 2026

  1. What GitHub issue is this PR addressing? Make sure that there is an accompanying issue to your PR.

    N/A

  2. Fill out the following pre-review checklist:

    • I am adding a new automated test(s) to verify correctness of my new code
    • I am adding new logging messages
    • I am adding new credentials
    • I am adding a new dependency (file filtering in GHA)
  3. Please describe how your code solves the related issue.

Adjust workflows:

  • Remove unused workflows (please see create_req_files.yml; wasn't sure if it's used or not)
  • Adjust testing workflow
    • Remove push event as we're mostly concerned about tests on PRs rather than on pushed changes
    • Change workflow_dispatch parameters (previous ones were not used)
    • Add a new job that runs first and:
      • Extracts PR data and verifies if the pipeline was run with the correct SHA (should be equal to head)
      • Checks for files changed to determine if tests should be run or not
    • Adjust ifs for each job, so that it skips the work when it should
    • Adjust checkout action in each job, so that the correct code is chosen

Also, potentially whole jira workflows could be simplified to just one workflow that creates tickets.

@sfc-gh-jcieslak sfc-gh-jcieslak force-pushed the jcieslak/workflow-adjustments branch from 550daad to eeb7758 Compare March 11, 2026 12:57
@sfc-gh-jcieslak sfc-gh-jcieslak marked this pull request as ready for review March 11, 2026 13:00
@sfc-gh-jcieslak sfc-gh-jcieslak requested a review from a team as a code owner March 11, 2026 13:00
- uses: actions/checkout@v4
- name: Checkout Code
id: checkout_code
uses: actions/checkout@v4
Copy link
Collaborator

Choose a reason for hiding this comment

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

How about moving to latest version ?

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