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 Burkert cpotential #367

Merged
merged 11 commits into from
Mar 26, 2024
Merged

Added Burkert cpotential #367

merged 11 commits into from
Mar 26, 2024

Conversation

HSouch
Copy link
Contributor

@HSouch HSouch commented Mar 25, 2024

Describe your changes

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 Mar 25, 2024

The code formatter check is grumpy about two very silly whitespace things https://github.com/adrn/gala/actions/runs/8427009942/job/23076877836?pr=367 -- I think the suggested changes I just left will fix it?

HSouch and others added 2 commits March 25, 2024 17:24
Co-authored-by: Adrian Price-Whelan <adrianmpw@gmail.com>
Co-authored-by: Adrian Price-Whelan <adrianmpw@gmail.com>
@adrn adrn added this to the v1.9 milestone Mar 26, 2024
@adrn adrn merged commit 4321701 into adrn:main Mar 26, 2024
20 checks passed
@adrn
Copy link
Owner

adrn commented Mar 26, 2024

Thanks @HSouch !!

@adrn
Copy link
Owner

adrn commented Mar 26, 2024

PS: I just added you to the AUTHORS list :) https://github.com/adrn/gala/blob/main/AUTHORS.rst

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants