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

refactor!: move to axum #42

Merged
merged 66 commits into from
Nov 9, 2024
Merged

refactor!: move to axum #42

merged 66 commits into from
Nov 9, 2024

Conversation

simonsan
Copy link
Contributor

@simonsan simonsan commented Jan 22, 2024

Fixes #2
Fixes #23

TODO:

@simonsan simonsan added A-meta Area: Project wide C-refactor Category: Refactoring of already existing code labels Jan 22, 2024
simonsan and others added 24 commits January 23, 2024 08:29
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
Signed-off-by: simonsan <[email protected]>
@simonsan
Copy link
Contributor Author

It may make sense to factor out the tests and merge them first, when they run through also with the old code. Or make them run with the old code.

@simonsan
Copy link
Contributor Author

When merging this PR:

  • deactivate the nightly builds for the time being, so this version is not being delivered until stabilized

  • create a note in the Readme, that this version is currently unstable and should not be used outside a development context.

Signed-off-by: simonsan <[email protected]>
@simonsan
Copy link
Contributor Author

simonsan commented Nov 9, 2024

Let's merge this and work in another PR to get the typed_routing going and implement more tests.

@simonsan simonsan merged commit f624412 into main Nov 9, 2024
21 checks passed
@simonsan simonsan deleted the feature/axum branch November 9, 2024 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-meta Area: Project wide C-refactor Category: Refactoring of already existing code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reactivate audit workflow, when refactored to axum Replace tide with axum
3 participants