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

deprecate Integrator.run for integrator.call #355

Merged
merged 5 commits into from
Aug 22, 2024

Conversation

nstarman
Copy link
Contributor

@nstarman nstarman commented Jan 22, 2024

Not sure if you were interested in this for gala. If you are, I'll finish the checklist / prune some unneeded lint changes that snuck in here.

Checklist

  • Did you add tests?
  • Did you add documentation for your changes?
  • Did you reference any relevant issues?
  • Did you add a changelog entry? (see CHANGES.rst)
  • Are the CI tests passing?
  • Is the milestone set?

@adrn
Copy link
Owner

adrn commented Feb 1, 2024

Yes, I think it makes sense to bring the API's closer before we all switch over to galax :). Thanks!

@nstarman
Copy link
Contributor Author

nstarman commented Feb 2, 2024

Added a test and changelog.

@adrn adrn added this to the v1.9 milestone Feb 3, 2024
@adrn
Copy link
Owner

adrn commented Feb 3, 2024

No idea why the testpypi upload step is failing, but it must be unrelated to this PR.

Thanks!

Signed-off-by: nstarman <nstarman@users.noreply.github.com>
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
Signed-off-by: nstarman <nstarman@users.noreply.github.com>
@adrn adrn merged commit 66443cf into adrn:main Aug 22, 2024
17 checks passed
@adrn
Copy link
Owner

adrn commented Aug 22, 2024

Sorry for the long delay on this @nstarman -- it fell off my radar! Thanks for this.

@nstarman nstarman deleted the deprecate-integrator-run branch August 22, 2024 15:46
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