Skip to content

Add reusable CI workflow for package build and test checks - #8

Open
Harikrishna-2016 wants to merge 24 commits into
mainfrom
feature/enable-ci-build-test-validation
Open

Harikrishna-2016 wants to merge 24 commits into
mainfrom
feature/enable-ci-build-test-validation

Conversation

@Harikrishna-2016

Copy link
Copy Markdown
Owner

What does this Pull Request accomplish?

  • Configures a pull request workflow targeting the main branch using a reusable CI workflow and NI Python Actions.
  • Executes the existing feature-level unit tests on Windows and Linux across Python 3.9-3.13.
  • Builds wheel and source distribution ("sdist") packages using Poetry.
  • Fails CI on build or test failures.
  • Publishes results as pull request checks.

Why should this Pull Request be merged?

  • Enables full CI with build and test for the nisyscfg-python repository
  • Detects build and test failures automatically before merging.
  • Improves package reliability across the supported platforms and Python versions.

What testing has been done?

  • Verified the PR workflow triggers on pull requests targeting main.
  • Verified the existing unit tests run on Windows and Linux for Python 3.9-3.13.
  • Verified the wheel and sdist packages build successfully using Poetry.
  • Verified the results are published as pull request checks.

@github-actions

github-actions Bot commented Oct 15, 2026 •

Copy link
Copy Markdown

Test Results

 10 files   10 suites   18s ⏱️
 83 tests  83 ✅ 0 💤 0 ❌
780 runs  780 ✅ 0 💤 0 ❌

Results for commit 0c8d7d3.

♻️ This comment has been updated with latest results.

@Harikrishna-2016

Harikrishna-2016 commented Oct 15, 2026 •

Copy link
Copy Markdown
Owner Author

Hi,

To keep this PR focused on the reusable CI workflow integration and reduce the risk of introducing unrelated changes, the following items are intentionally excluded from the current scope:

  • Enabling the Analyze Python Project action and Bandit security checks.
  • Remove the analyze_package.yml workflow and use the lint action directly for authentication.
  • Migrating the existing PyPI publishing workflow, including authentication and Trusted Publishing changes.
  • Implementing documentation generation and artifact upload.

These items require additional validation and configuration, so they will be addressed separately in follow-up PRs.

Thanks!

sourabhutaniCG added a commit that referenced this pull request Oct 23, 2026
- [x] This contribution adheres to
[CONTRIBUTING.md](https://github.com/tkrebes/nisyscfg-python/blob/master/CONTRIBUTING.md).


**What does this Pull Request accomplish?**
- Configures a pull request workflow targeting the `main` branch using a
reusable CI workflow and NI Python Actions.
- Executes the existing feature-level unit tests on Windows and Linux
across Python **3.9-3.13.**
- Builds wheel and source distribution ("sdist") packages using Poetry.
- Fails CI on build or test failures.
- Publishes results as pull request checks.


**Why should this Pull Request be merged?**
- Enables full CI with build and test for the **nisyscfg-python**
repository
- Detects build and test failures automatically before merging.
- Improves package reliability across the supported platforms and Python
versions.

**What testing has been done?**
- Verified the PR workflow triggers on pull requests targeting `main`.
- Verified the existing unit tests run on Windows and Linux for Python
**3.9-3.13.**
- Verified the wheel and sdist packages build successfully using Poetry.
- Verified the results are published as pull request checks.

---------

Co-authored-by: sourabhutaniCG <sourabh.bhutani@capgemini.com>
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