Skip to content

Conversation

@dixonjoel
Copy link
Collaborator

@dixonjoel dixonjoel commented Sep 11, 2025

What does this Pull Request accomplish?

Add a analyze-project action that can be re-used in other repos.

Why should this Pull Request be merged?

Reuse the .yml files instead of copying them around to every repo.

What testing has been done?

Tested by calling the new action from this branch in datastore-python and nipanel-python repos

Checks from using this action in nipanel:
https://github.com/ni/nipanel-python/actions/runs/17921957525

Checks from using this action in datastore-python:
https://github.com/ni/datastore-python/actions/runs/17921449681

@dixonjoel

This comment was marked as outdated.

@bkeryan
Copy link
Collaborator

bkeryan commented Sep 12, 2025

FYI @dixonjoel I'm splitting the README.md in preparation for adding workflows and/or more actions
#25

@dixonjoel dixonjoel changed the title Add check_analyzers Add analyze-project to validate a Python project Sep 16, 2025
@dixonjoel dixonjoel changed the title Add analyze-project to validate a Python project Add analyze-project composite action to validate a Python project Sep 18, 2025
@dixonjoel dixonjoel requested a review from bkeryan September 24, 2025 14:27
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

@dixonjoel dixonjoel requested a review from bkeryan September 24, 2025 17:34
@dixonjoel
Copy link
Collaborator Author

Approved with suggestions

@bkeryan Sorry, it was still showing you as Requested Changes, so I reset you again.

@mshafer-NI mshafer-NI closed this Sep 24, 2025
@mshafer-NI mshafer-NI reopened this Sep 24, 2025
@dixonjoel dixonjoel merged commit 1b90235 into main Sep 24, 2025
222 checks passed
@dixonjoel dixonjoel deleted the users/jdixon/add-common-workflows branch September 24, 2025 20:03
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.

5 participants