Skip to content
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.

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

Added Scripts for Deployment Testing to TestPypi #236

Merged
merged 23 commits into from
Jun 16, 2023
Merged

Conversation

shahidee44
Copy link
Collaborator

@shahidee44 shahidee44 commented May 23, 2023

Description

  • Fixed metapackage deployment issue.
  • Added/Updated scripts for deployment testing of individual plugins to TestPypi
  • Updated scripts for local install and testing for individual plugins
  • Versions are now caveated with -rcX to represent pre-release. (These versions can only be downloaded using the --pre flag)
  • openqaoa metapackage now installs its dependencies from Pypi.
  • Fixed test_dev.yml workflow and Dockerfile to use new Makefile commands.

Checklist

  • I have performed a self-review of my code.
  • I have commented my code and used numpy-style docstrings
  • My changes generate no new warnings
  • I have added/updated tests to make sure bugfix/feature works.
  • New and existing unit tests pass locally with my changes.

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Name the new unit-tests that you have added along with this change.

@shahidee44
Copy link
Collaborator Author

openqaoa metapackage renamed to openqaoa-meta for testing purposes. (To change back before merging with dev)

@shahidee44 shahidee44 requested a review from vishal-ph June 13, 2023 05:59
@vishal-ph vishal-ph merged commit 810755d into dev Jun 16, 2023
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