Skip to content

Conversation

hbalty
Copy link
Contributor

@hbalty hbalty commented Aug 11, 2025

No description provided.

@revu-bot revu-bot bot requested a review from revu-bot August 11, 2025 09:02
Copy link
Collaborator

@revu-bot revu-bot left a comment

Choose a reason for hiding this comment

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

An error occurred: ## ⚠️ PR Review Skipped

Issues Found

1. This PR contains files that exceed the size limit: 'yarn.lock' (3859 lines of changes), which exceeds the limit of 2000. The limit is 2000 lines per file.

Suggestion: Consider refactoring large changes into smaller, more focused modifications. Large file changes are harder to review and understand.

PR Metrics

  • Total files changed: 14
  • Reviewable files: 14
  • Diff size: 4309 lines
  • Documentation files: 0
  • Largest file change: 3859 lines
  • Addition/Deletion ratio: 20.58

This validation helps ensure the bot focuses on PRs where automated review provides the most value. You can adjust these limits in your .revu.yml configuration file.

Revu logs

@hbalty hbalty force-pushed the feat-migrating-portail-admin-v18 branch from d6d0d12 to 99f3523 Compare August 11, 2025 14:51
@hbalty hbalty temporarily deployed to build-review-auto August 11, 2025 14:51 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto August 11, 2025 14:51 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto August 11, 2025 14:51 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto August 11, 2025 14:51 — with GitHub Actions Inactive
@codecov-commenter
Copy link

codecov-commenter commented Aug 11, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.93%. Comparing base (04e8920) to head (ca9bc2c).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           master    #3847    +/-   ##
========================================
  Coverage   65.93%   65.93%            
========================================
  Files         971      971            
  Lines       16256    16256            
  Branches     2251     2251            
========================================
  Hits        10719    10719            
+ Misses       5332     5191   -141     
- Partials      205      346   +141     

see 60 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 04e8920...ca9bc2c. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@hbalty hbalty force-pushed the feat-migrating-portail-admin-v18 branch from 99f3523 to f7adcab Compare August 11, 2025 20:00
@hbalty hbalty temporarily deployed to build-review-auto August 11, 2025 20:00 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto August 11, 2025 20:00 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto August 11, 2025 20:00 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto August 11, 2025 20:00 — with GitHub Actions Inactive
Copy link

socket-security bot commented Aug 11, 2025

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License

View full report

Copy link

socket-security bot commented Aug 11, 2025

All alerts resolved. Learn more about Socket for GitHub.

This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored.

View full report

@hbalty hbalty force-pushed the feat-migrating-portail-admin-v18 branch from f7adcab to 7c8bb1b Compare August 11, 2025 21:38
@hbalty hbalty force-pushed the feat-migrating-portail-admin-v18 branch from 7c8bb1b to 58291be Compare August 11, 2025 21:38
@hbalty hbalty temporarily deployed to build-review-auto August 11, 2025 21:39 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto August 11, 2025 21:39 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto August 11, 2025 21:39 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto August 11, 2025 21:39 — with GitHub Actions Inactive
@hbalty hbalty requested a review from revu-bot August 12, 2025 10:52
Copy link
Collaborator

@revu-bot revu-bot left a comment

Choose a reason for hiding this comment

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

An error occurred: ## ⚠️ PR Review Skipped

Issues Found

1. This PR contains files that exceed the size limit: 'yarn.lock' (3859 lines of changes), which exceeds the limit of 2000. The limit is 2000 lines per file.

Suggestion: Consider refactoring large changes into smaller, more focused modifications. Large file changes are harder to review and understand.

PR Metrics

  • Total files changed: 13
  • Reviewable files: 13
  • Diff size: 4299 lines
  • Documentation files: 0
  • Largest file change: 3859 lines
  • Addition/Deletion ratio: 20.74

This validation helps ensure the bot focuses on PRs where automated review provides the most value. You can adjust these limits in your .revu.yml configuration file.

Revu logs

@tokenbureau
Copy link

tokenbureau bot commented Aug 14, 2025

🎉 Deployment for commit 28bdcbe :

Ingresses
Docker images
  • 📦 docker pull harbor.fabrique.social.gouv.fr/domifa/domifa/backend:sha-28bdcbeb7895900bbf37642bb121e0ff22ffd28d
  • 📦 docker pull harbor.fabrique.social.gouv.fr/domifa/domifa/frontend:sha-28bdcbeb7895900bbf37642bb121e0ff22ffd28d
  • 📦 docker pull harbor.fabrique.social.gouv.fr/domifa/domifa/portail-admins:sha-28bdcbeb7895900bbf37642bb121e0ff22ffd28d
  • 📦 docker pull harbor.fabrique.social.gouv.fr/domifa/domifa/portail-usagers:sha-28bdcbeb7895900bbf37642bb121e0ff22ffd28d
Debug

@hbalty hbalty force-pushed the feat-migrating-portail-admin-v18 branch from 28bdcbe to 7972e9b Compare August 14, 2025 15:32
@hbalty hbalty temporarily deployed to build-review-auto August 14, 2025 15:32 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto August 14, 2025 15:32 — with GitHub Actions Inactive
@hbalty hbalty temporarily deployed to build-review-auto August 14, 2025 15:32 — with GitHub Actions Inactive
@hbalty hbalty had a problem deploying to build-review-auto August 14, 2025 15:32 — with GitHub Actions Failure
@pYassine pYassine force-pushed the feat-migrating-portail-admin-v18 branch from 7972e9b to 183bb92 Compare August 18, 2025 14:42
@pYassine pYassine temporarily deployed to build-review-auto August 18, 2025 14:42 — with GitHub Actions Inactive
@pYassine pYassine temporarily deployed to build-review-auto August 18, 2025 14:42 — with GitHub Actions Inactive
@pYassine pYassine temporarily deployed to build-review-auto August 18, 2025 14:42 — with GitHub Actions Inactive
@pYassine pYassine temporarily deployed to build-review-auto August 18, 2025 14:42 — with GitHub Actions Inactive
Copy link

@pYassine pYassine merged commit 5b6121e into master Aug 18, 2025
18 of 19 checks passed
@pYassine pYassine deleted the feat-migrating-portail-admin-v18 branch August 18, 2025 14:56
@tokenbureau
Copy link

tokenbureau bot commented Aug 18, 2025

🎉 This PR is included in version 2.217.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@tokenbureau tokenbureau bot added the released label Aug 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants