Skip to content

fix(airline): Air France and KLM Royal Dutch Airlines #3440

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Mar 17, 2025

Conversation

chimurai
Copy link
Contributor

@chimurai chimurai commented Mar 16, 2025

fixes airline: Air France (AF) and KLM Royal Dutch Airlines (KL)

Air France-KLM Group is the French-Dutch multinational airline holding company

@chimurai chimurai requested a review from a team as a code owner March 16, 2025 19:17
Copy link

netlify bot commented Mar 16, 2025

Deploy Preview for fakerjs ready!

Name Link
🔨 Latest commit fc1d4dc
🔍 Latest deploy log https://app.netlify.com/sites/fakerjs/deploys/67d87262d9198c0008ee65e1
😎 Deploy Preview https://deploy-preview-3440.fakerjs.dev
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@chimurai chimurai changed the title fix(airline): Air France and KLM fix(airline): Air France and KLM Royal Dutch Airlines Mar 16, 2025
@xDivisionByZerox xDivisionByZerox added this to the vAnytime milestone Mar 16, 2025
@xDivisionByZerox xDivisionByZerox added p: 1-normal Nothing urgent c: locale Permutes locale definitions m: airline Something is referring to the airline module labels Mar 16, 2025
Copy link

codecov bot commented Mar 16, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 99.97%. Comparing base (fef0ad7) to head (fc1d4dc).
Report is 1 commits behind head on next.

Additional details and impacted files
@@           Coverage Diff           @@
##             next    #3440   +/-   ##
=======================================
  Coverage   99.97%   99.97%           
=======================================
  Files        2817     2817           
  Lines      217493   217494    +1     
  Branches      950      949    -1     
=======================================
+ Hits       217438   217439    +1     
  Misses         55       55           
Files with missing lines Coverage Δ
src/locales/en/airline/airline.ts 100.00% <100.00%> (ø)
🚀 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.

Copy link
Member

@xDivisionByZerox xDivisionByZerox left a comment

Choose a reason for hiding this comment

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

Thankt you for your contribution. ❤️

After some quick research, I was able to verify the suggested change.

@xDivisionByZerox xDivisionByZerox merged commit 8a2d168 into faker-js:next Mar 17, 2025
23 checks passed
@chimurai chimurai deleted the airline-fix branch March 17, 2025 19:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c: locale Permutes locale definitions m: airline Something is referring to the airline module p: 1-normal Nothing urgent
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants