Skip to content

Removes Persola as a submodule of Deepiri Platform, team submodule commands, and team dev enviroments#218

Open
Patrik-Corri wants to merge 30 commits into
mainfrom
patrik_corri-dev
Open

Removes Persola as a submodule of Deepiri Platform, team submodule commands, and team dev enviroments#218
Patrik-Corri wants to merge 30 commits into
mainfrom
patrik_corri-dev

Conversation

@Patrik-Corri
Copy link
Copy Markdown
Member

IMPORTANT:

  • PR must be opened from your personal branch → dev
  • You must tag @Team-Deepiri/support-team
  • You must update Plaky to "Needs QA"
  • Never move a task to "Done" (Done = production release only)

Description

Briefly explain what this PR does and why.

Include:

  • Related Issue number
  • Plaky feature name
  • Component, feature, or system affected
  • Purpose of change (feature, bug fix, improvement, refactor, security, etc.)

Changes

List the most important updates in this PR:

Be specific. Include:

  • New or updated functions, services, components, or scripts
  • Refactoring or structural improvements
  • Dependency or configuration changes
  • Any significant implementation details

Related

  • Issue:
  • Plaky:
  • Related PRs (optional):

Testing

Explain how you verified your changes and how to test your feature:

Additional testing details:


Important Notes (Optional)

  • Known limitations:
  • Blockers:
  • CI/CD issues unrelated to this PR:
  • Dependencies required for testing:

Workflow Checklist (Required)

  • Branch is up to date with dev
  • PR is from your branch → dev (no longer directly into main)
  • PR title follows convention (feat:, fix:, refactor:, etc.)
  • Plaky feature/bug name included above
  • Tagged @Team-Deepiri/support-team
  • Plaky task moved to "Needs QA"

Review Requests

@Team-Deepiri/support-team

@jrb00013

Patrik-Corri and others added 24 commits March 7, 2026 20:40
…ckerfiles

Normalize load-k8s-env.sh, docker-entrypoint.sh, prisma-baseline.sh (and ops/k8s copies) to Unix LF.

Drop RUN sed CRLF fixes from service Dockerfiles; chmod-only after COPY.

Add .gitattributes eol=lf for shared/scripts and ops/k8s *.sh.

Made-with: Cursor
Co-authored-by: Cursor <cursoragent@cursor.com>
@jrb00013 jrb00013 changed the title Patrik corri dev Removes Persola from Deepiri Platform, team submodule commands, and team dev enviroments May 5, 2026
@jrb00013 jrb00013 changed the title Removes Persola from Deepiri Platform, team submodule commands, and team dev enviroments Removes Persola as a submodule of Deepiri Platform, team submodule commands, and team dev enviroments May 5, 2026
@jrb00013
Copy link
Copy Markdown
Member

jrb00013 commented May 5, 2026

@Patrik-Corri homie this looks like an old branch, why didn't you pull or implement off a branch based off main

@Patrik-Corri
Copy link
Copy Markdown
Member Author

i pulled from main before making changes, it shouldn't be old.

@Patrik-Corri
Copy link
Copy Markdown
Member Author

@jrb00013

Comment thread platform-services/backend/deepiri-realtime-gateway/Dockerfile Outdated
Comment thread docker-compose.dev.yml Outdated
@Patrik-Corri
Copy link
Copy Markdown
Member Author

don't know why the dockerfiles were edited, fixed now @jrb00013

Copy link
Copy Markdown
Member

@jrb00013 jrb00013 left a comment

Choose a reason for hiding this comment

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

You also need to delete the persola itself

@jrb00013
Copy link
Copy Markdown
Member

jrb00013 commented May 9, 2026

@Patrik-Corri resolve conflicts as well

Patrik-Corri and others added 3 commits May 9, 2026 18:49
Co-authored-by: Cursor <cursoragent@cursor.com>
…submodules, remove diri-persola

Co-authored-by: Cursor <cursoragent@cursor.com>
Comment thread docker-compose.dev.yml
- deepiri-dev-network

# Persola - Personalized Agentic Framework
# Profile: cyrex — only active when started with: docker compose --profile cyrex up
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Take out the persola from the docker compose dev yml

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.

3 participants