Skip to content

Conversation

@mjohanse-emr
Copy link
Collaborator

@mjohanse-emr mjohanse-emr commented Jun 24, 2025

What does this Pull Request accomplish?

Add CI and PR workflows that perform a subset of the steps they will eventually perform. Right now, they only run the check_<package>.yml workflows.

The rest of the workflow files have been left out for now (check docs, unit tests, etc.). I'll add them back in when I start using them from CI.yml.

Why should this Pull Request be merged?

Initial steps for AB#3166738

What testing has been done?

PR/CI checks pass.

Signed-off-by: Michael Johansen <[email protected]>

Fix the working directory setting.

Signed-off-by: Michael Johansen <[email protected]>

Specify working directory for pyright.

Signed-off-by: Michael Johansen <[email protected]>

I put the input in the wrong spot.

Signed-off-by: Michael Johansen <[email protected]>

Remove unused workflows. They will be added in later.

Signed-off-by: Michael Johansen <[email protected]>
@mjohanse-emr mjohanse-emr force-pushed the users/mjohanse/workflows branch from b00a993 to 9a9dbb2 Compare June 24, 2025 18:30
@mjohanse-emr mjohanse-emr marked this pull request as ready for review June 24, 2025 18:32
Copy link
Collaborator

@bkeryan bkeryan left a comment

Choose a reason for hiding this comment

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

Approved with suggestions

@mjohanse-emr mjohanse-emr merged commit 119ce90 into main Jun 24, 2025
2 checks passed
@mjohanse-emr mjohanse-emr deleted the users/mjohanse/workflows branch June 24, 2025 19:36
on:
workflow_call:
inputs:
package_name:
Copy link
Collaborator

Choose a reason for hiding this comment

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

Action parameters typically use kebab-case.

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.

3 participants