Skip to content

Development#812

Merged
NishaSharma14 merged 3 commits into
mainfrom
development
Jun 8, 2026
Merged

Development#812
NishaSharma14 merged 3 commits into
mainfrom
development

Conversation

@NishaSharma14

Copy link
Copy Markdown
Contributor

No description provided.

NishaSharma14 and others added 3 commits June 1, 2026 14:14
* chore(deps): resolve composer security advisories

Bump locked dependencies to clear composer audit, including Filament
4.11.6, Laravel 12.61, Symfony security patches, PhpSpreadsheet 1.30.5,
and related transitive packages.

* chore(filament): republish assets after dependency upgrade

Refresh published Filament 4 and plugin frontend assets to match the
updated composer lockfile.

* style: apply Laravel Pint across application code

Normalize imports and class references in app layer (HTTP kernel,
Filament resources, REST API, jobs, events, models) per Pint rules.

* style: normalize imports in config and routes

Apply Pint class import conventions to published config files and
api routes without changing configuration values.

* style: normalize imports in factories and migrations

Replace FQCN references with imported class names in database
factories and migrations for consistency with Pint output.

* fix(models): add sampleLocations relationship on Molecule

Define the belongsToMany relationship via molecule_organism so Filament
and PHPStan can resolve sample location access on molecules.

* fix(phpstan): resolve static analysis errors in publish and Filament code

Remove stale phpstan-ignore on PublishMolecules and narrow collection
updates with explicit null checks. Type the organism molecules bulk
action repeater default using Molecule and the sampleLocations relation.
@codecov-commenter

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 6.66667% with 56 lines in your changes missing coverage. Please review.
✅ Project coverage is 6.13%. Comparing base (e7e52fa) to head (4223a58).
⚠️ Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
...urce/RelationManagers/MoleculesRelationManager.php 0.00% 19 Missing ⚠️
app/Models/Molecule.php 0.00% 12 Missing ⚠️
app/Rest/Resources/ReportResource.php 0.00% 9 Missing ⚠️
app/Rest/Resources/MoleculeResource.php 0.00% 6 Missing ⚠️
app/Rest/Resource.php 0.00% 5 Missing ⚠️
app/Jobs/ImportPubChemAuto.php 0.00% 2 Missing ⚠️
app/Http/Controllers/Auth/SocialController.php 0.00% 1 Missing ⚠️
app/Jobs/ClassifyMoleculeAuto.php 0.00% 1 Missing ⚠️
app/Jobs/ProcessEntryBatch.php 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##              main    #812      +/-   ##
==========================================
- Coverage     6.14%   6.13%   -0.02%     
- Complexity    1736    1737       +1     
==========================================
  Files          227     227              
  Lines         8945    8961      +16     
==========================================
  Hits           550     550              
- Misses        8395    8411      +16     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 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.

@NishaSharma14 NishaSharma14 merged commit e4a4f53 into main Jun 8, 2026
6 checks passed
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