Skip to content

[ES-2881] added verify for github action#145

Open
zesu22 wants to merge 1 commit intomosip:developfrom
Infosys:task/ES-2881_1
Open

[ES-2881] added verify for github action#145
zesu22 wants to merge 1 commit intomosip:developfrom
Infosys:task/ES-2881_1

Conversation

@zesu22
Copy link
Contributor

@zesu22 zesu22 commented Mar 6, 2026

Summary by CodeRabbit

  • Chores
    • Updated package configuration to include repository directory metadata.
    • Added a new build verification script to the package configuration.

Signed-off-by: Zeeshan Mehboob <zeeshan.mehboob@infosys.com>
@coderabbitai
Copy link

coderabbitai bot commented Mar 6, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 06a9ee27-6d78-41b2-a5ea-4dce938546e8

📥 Commits

Reviewing files that changed from the base of the PR and between 2d54cc4 and 5736499.

📒 Files selected for processing (1)
  • sign-in-with-esignet/package.json

Walkthrough

Added repository.directory field and a verify npm script to the sign-in-with-esignet/package.json file. The verify script is configured to run the build command.

Changes

Cohort / File(s) Summary
Package Configuration
sign-in-with-esignet/package.json
Added repository.directory field pointing to package location and new verify npm script that executes the build command.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 Hop, hop, hooray! 🎉
New scripts and directories in place,
Configuration blooms with grace,
Build verification finds its space,
This package now runs at faster pace!

🚥 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 mentions adding a 'verify' script for GitHub action, which directly corresponds to the new npm verify script added in the changeset.
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
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

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