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

Add is_incoming and is_outgoing to exports #60

Merged
merged 1 commit into from
May 8, 2024

Conversation

AntonReinhard
Copy link
Member

They were missing from the exports so I added them.

@AntonReinhard AntonReinhard self-assigned this May 3, 2024
@AntonReinhard
Copy link
Member Author

I added a PR in QED.jl to fix the integration test generator which is currently broken here: QEDjl-project/QuantumElectrodynamics.jl#31

@szabo137
Copy link
Member

szabo137 commented May 3, 2024

Thanks for this! I see the issue with the integration tests. I think we need to wait until QEDjl-project/QuantumElectrodynamics.jl#31 is merged. Then we can trigger the test again and proceed with this one.

(Btw. Many thanks for providing the fix for the integration test bug in the first place 👌)

@AntonReinhard
Copy link
Member Author

@szabo137 If there are no more problems, please approve and merge

@szabo137 szabo137 merged commit 0b1fde0 into QEDjl-project:dev May 8, 2024
4 checks passed
@szabo137 szabo137 added this to the Release-next milestone May 14, 2024
@AntonReinhard AntonReinhard deleted the export_missing branch June 24, 2024 08:03
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