Skip to content

Conversation

pYassine
Copy link
Contributor

No description provided.

@revu-bot revu-bot bot requested a review from revu-bot August 11, 2025 15:09
@pYassine pYassine marked this pull request as ready for review August 11, 2025 15:11
@pYassine pYassine requested a review from a team as a code owner August 11, 2025 15:11
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: 400 {"type":"error","error":{"type":"invalid_request_error","message":"prompt is too long: 204118 tokens > 200000 maximum"}}

Revu logs

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
Added@​angular/​language-service@​17.3.121001006897100
Added@​types/​file-saver@​2.0.71001007876100
Addedng-lint-staged@​12.0.4891008576100
Addedfile-saver@​2.0.510010010079100
Added@​popperjs/​core@​2.11.810010010079100
Addedzone.js@​0.14.1010010010081100

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

@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

❌ Patch coverage is 30.76923% with 9 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.78%. Comparing base (177fb1f) to head (d0c29ae).
⚠️ Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
...nd/src/util/sentry/AppSentryInterceptor.service.ts 18.18% 8 Missing and 1 partial ⚠️
❗ 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    #3849      +/-   ##
==========================================
+ Coverage   65.70%   65.78%   +0.08%     
==========================================
  Files         970      970              
  Lines       16238    16220      -18     
  Branches     2245     2237       -8     
==========================================
+ Hits        10669    10671       +2     
+ Misses       5224     5208      -16     
+ Partials      345      341       -4     
Files with missing lines Coverage Δ
packages/backend/src/app.module.ts 100.00% <100.00%> (ø)
...nd/src/util/sentry/AppSentryInterceptor.service.ts 27.77% <18.18%> (+11.98%) ⬆️

... and 3 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 177fb1f...d0c29ae. 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.

@pYassine pYassine temporarily deployed to build-review-auto August 11, 2025 15:41 — with GitHub Actions Inactive
@pYassine pYassine temporarily deployed to build-review-auto August 11, 2025 15:41 — with GitHub Actions Inactive
@pYassine pYassine temporarily deployed to build-review-auto August 11, 2025 15:41 — with GitHub Actions Inactive
@pYassine pYassine force-pushed the feat-upgrade-sentry branch from c0f6eab to d0c29ae Compare August 11, 2025 15:52
@pYassine pYassine temporarily deployed to build-review-auto August 11, 2025 15:52 — with GitHub Actions Inactive
@pYassine pYassine temporarily deployed to build-review-auto August 11, 2025 15:52 — with GitHub Actions Inactive
@pYassine pYassine temporarily deployed to build-review-auto August 11, 2025 15:52 — with GitHub Actions Inactive
@pYassine pYassine temporarily deployed to build-review-auto August 11, 2025 15:52 — with GitHub Actions Inactive
Copy link

@tokenbureau
Copy link

tokenbureau bot commented Aug 11, 2025

🎉 Deployment for commit d0c29ae :

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

@pYassine pYassine merged commit 640370a into master Aug 11, 2025
19 checks passed
@pYassine pYassine deleted the feat-upgrade-sentry branch August 11, 2025 16:16
@tokenbureau
Copy link

tokenbureau bot commented Aug 11, 2025

🎉 This PR is included in version 2.212.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@tokenbureau tokenbureau bot added the released label Aug 11, 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