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

Remove iron integration #1210

Merged
merged 3 commits into from
Nov 10, 2023
Merged

Remove iron integration #1210

merged 3 commits into from
Nov 10, 2023

Conversation

tyranron
Copy link
Member

@tyranron tyranron commented Nov 9, 2023

This PR removes juniper_iron crate from workspace, and thus, support for the iron crate for future juniper versions.

Motivation

iron crate seems to become unmaintained and unsupported for a while, confirmed by its authors:

It also depends on quite old versions, introducing regular problems to CI.

Support criteria

Once iron crate becomes actively maintained again and supports latest versions of its dependencies, it would be easy to restore juniper_iron crate back into the workspace and maintain it.

@tyranron tyranron added k::documentation Related to project documentation k::api Related to API (application interface) lib::iron Related to `iron` crate integration k::example Related to usage examples labels Nov 9, 2023
@tyranron tyranron added this to the 0.16.0 milestone Nov 9, 2023
@tyranron tyranron self-assigned this Nov 9, 2023
@tyranron tyranron added the rollback Rollback of some changes label Nov 9, 2023
@tyranron
Copy link
Member Author

tyranron commented Nov 9, 2023

@LegNeato before merging this, I would like to hear any comments of yours on this, if you have ones to share.

@LegNeato
Copy link
Member

I think this is the right call! I doubt anyone is using iron anymore, and if they are they can use old versions or add a pr and sign up to support it themselves.

@LegNeato LegNeato merged commit d7103e2 into master Nov 10, 2023
172 checks passed
@LegNeato LegNeato deleted the remove-iron branch November 10, 2023 03:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
k::api Related to API (application interface) k::documentation Related to project documentation k::example Related to usage examples lib::iron Related to `iron` crate integration rollback Rollback of some changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants