Skip to content

Added a github action to automatically test code on PRs and commits #38

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 10 commits into from
Mar 31, 2025
Merged

Added a github action to automatically test code on PRs and commits #38

merged 10 commits into from
Mar 31, 2025

Conversation

@amit-sharma
Copy link
Member

@amit-sharma amit-sharma commented Mar 29, 2025 •
edited
Loading

  • Creating automatic ci so that test run on each PR
  • Upgraded the package versions in pyproject
  • Removed google and other imports from notebooks
  • ignoring notebooks for testing right now since they require a openai key

@amit-sharma
Create python-package.yml
5f48587 
Signed-off-by: Amit Sharma <[email protected]>
@amit-sharma
updated pyproject to support python3.9+ to enable support for guidance
4e0bf53 
Signed-off-by: Amit Sharma <[email protected]>
@amit-sharma
added latest versions of deps
80cb5c6 
Signed-off-by: Amit Sharma <[email protected]>
@amit-sharma
upgraded setuptools and wheel
3aef1cc 
Signed-off-by: Amit Sharma <[email protected]>
@amit-sharma
upgraded poetry
ef584dd 
Signed-off-by: Amit Sharma <[email protected]>
@amit-sharma
downgrade version of pytest
39b2f35 
Signed-off-by: Amit Sharma <[email protected]>
@amit-sharma
updated folder for pytest
8dd8853 
Signed-off-by: Amit Sharma <[email protected]>
@amit-sharma
updated folder for pytest
09ef0b1 
Signed-off-by: Amit Sharma <[email protected]>
@amit-sharma
ignore notebook tests
9e8cfbe 
Signed-off-by: Amit Sharma <[email protected]>
@amit-sharma amit-sharma changed the title Create python-package.yml Added a github action to automatically test code on PRs and commits Mar 29, 2025
emrekiciman
emrekiciman reviewed Mar 31, 2025
View reviewed changes
@amit-sharma
newer setup-python v5
0cb0b0d 
Signed-off-by: Amit Sharma <[email protected]>
@amit-sharma amit-sharma requested a review from emrekiciman March 31, 2025 07:41
emrekiciman
emrekiciman approved these changes Mar 31, 2025
View reviewed changes
Copy link
Member

@emrekiciman emrekiciman left a comment

Choose a reason for hiding this comment

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

lgtm. Thanks Amit!

@emrekiciman emrekiciman merged commit 90133c8 into main Mar 31, 2025
4 checks passed
@emrekiciman emrekiciman deleted the add-ci branch March 31, 2025 08:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@emrekiciman emrekiciman emrekiciman approved these changes

@grace-sng7 grace-sng7 Awaiting requested review from grace-sng7

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@amit-sharma @emrekiciman