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

cleanup container volume names, delete unused items #68

Merged
merged 8 commits into from
Dec 14, 2024
Merged

Conversation

manoranjith
Copy link
Collaborator

  • chore: remove unused scripts and asc directory
  • chore: override project, network name for compose
  • chore: cleanup fhevm, copro related names
  • chore: cleanup kms related names
  • chore: explicit container names to avoid -1 suffix
  • chore: remove completed TODOs

- use "fhevm-" prefix consistently for all components.
- use "setup-" for setup related components.
- use no prefix for gateway as it connects fhevm and kms parts.
- use "kms-" prefix consistently for all kms components.
- use "setup-" for setup related components.
@leventdem
Copy link
Collaborator

I have created another PR based on this one with minor improvements. Let's merge it before : #70

chore: clean a bit more some files, update docker service names everywhere, …
Copy link
Collaborator

@leventdem leventdem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@manoranjith manoranjith merged commit c169ba1 into main Dec 14, 2024
1 check passed
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