Skip to content

feat(eps): DOMA-12784 server side pdf check#7418

Open
vovaaxeapolla wants to merge 1 commit intomainfrom
feat/eps/DOMA-12784/server-side-pdf-check
Open

feat(eps): DOMA-12784 server side pdf check#7418
vovaaxeapolla wants to merge 1 commit intomainfrom
feat/eps/DOMA-12784/server-side-pdf-check

Conversation

@vovaaxeapolla
Copy link
Copy Markdown
Contributor

@vovaaxeapolla vovaaxeapolla commented Apr 2, 2026

Summary by CodeRabbit

  • Chores
    • Updated the eps submodule to a newer version.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 2f3cf183-ed67-49b6-aa1a-da4c522014bb

📥 Commits

Reviewing files that changed from the base of the PR and between 575924e and 6d8b81f.

📒 Files selected for processing (1)
  • apps/eps

📝 Walkthrough

Walkthrough

The apps/eps submodule pointer was updated to reference a different commit hash. This change modifies the exact version of the subproject that gets checked out, transitioning from one specific commit to another without alterations to public entity declarations.

Changes

Cohort / File(s) Summary
Submodule Pointer Update
apps/eps
Updated submodule commit reference from 66890896f711800a7401020a449724542dafec20 to 0927d209d82fb003aa39765a2ddbe77b7bae3648.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A carrot update, so simple and small,
A submodule twist, no changes at all,
One hash swapped for another with care,
The bunny commits with a hop and a flair! 🥕

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'feat(eps): DOMA-12784 server side pdf check' accurately describes the main change—updating the eps submodule to implement server-side PDF checking, as evidenced by the source branch name and ticket reference.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/eps/DOMA-12784/server-side-pdf-check

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 2, 2026

Copy link
Copy Markdown
Contributor

@dkoviazin dkoviazin left a comment

Choose a reason for hiding this comment

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

Will try to use template (HTML, JSON, WORD) to render with carbone.io

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants