Skip to content

Remove Celery and its related tasks#741

Merged
yeasy merged 3 commits intohyperledger-cello:mainfrom
dodo920306:refactor/remove-celery
Jan 12, 2026
Merged

Remove Celery and its related tasks#741
yeasy merged 3 commits intohyperledger-cello:mainfrom
dodo920306:refactor/remove-celery

Conversation

@dodo920306
Copy link
Contributor

@dodo920306 dodo920306 commented Dec 14, 2025

Summary

Remove Celery and its related tasks to simplify the project architecture

Changes

  • Remove api_engine/celery.py configuration file
  • Remove all Celery tasks under api/tasks/ directory
  • Update requirements.txt to remove Celery-related dependencies

Motivation

Celery is no longer used in the current architecture. Removing it reduces project complexity and dependencies.


🤖 This PR description was generated with AI assistance (Claude, Anthropic).

Signed-off-by: dodo920306 <dodo920306@gmail.com>
Signed-off-by: dodo920306 <dodo920306@gmail.com>
@dodo920306 dodo920306 changed the title Remove Celery and its related tasks to simplify the project architecture Remove Celery and its related tasks Dec 14, 2025
@dodo920306 dodo920306 mentioned this pull request Dec 14, 2025
@yeasy
Copy link
Contributor

yeasy commented Dec 28, 2025

Why CI fails?

@dodo920306
Copy link
Contributor Author

Why CI fails?

I don't actually know. I don't think I changed any thing that will cause the newman tests failed, but they failed.

Signed-off-by: dodo920306 <dodo920306@gmail.com>
@dodo920306
Copy link
Contributor Author

dodo920306 commented Jan 12, 2026

Why CI fails?

@yeasy It now passed. Can you merge this please 🙏?

@yeasy yeasy merged commit 27f4573 into hyperledger-cello:main Jan 12, 2026
3 checks passed
@dodo920306 dodo920306 deleted the refactor/remove-celery branch January 13, 2026 01:36
@dodo920306
Copy link
Contributor Author

dodo920306 commented Jan 13, 2026

@yeasy Do we have to use GitHub Pages? If not, can we turn it off? It failed to be deployed on the main branch, and I don't have the permission to change the settings of it.

image

I don't think we actually need this since we already have Read the Docs.

@yeasy
Copy link
Contributor

yeasy commented Jan 16, 2026

It seems defined in github actions.

@dodo920306
Copy link
Contributor Author

It seems defined in github actions.

What? No way. Our GitHub Actions are extremely simple.
https://github.com/hyperledger-cello/cello/tree/main/.github/workflows

However, it seems like the original failed GitHub Page deployment task disappeared automatically, so I guess that is fine.

@yeasy
Copy link
Contributor

yeasy commented Jan 17, 2026

Now it's closed.

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