Skip to content

fix: improve runway parsing #783

Merged
mivek merged 1 commit intomivek:mainfrom
f-loris:improvement/runway-parsing
Apr 8, 2026
Merged

fix: improve runway parsing #783
mivek merged 1 commit intomivek:mainfrom
f-loris:improvement/runway-parsing

Conversation

@f-loris
Copy link
Copy Markdown
Contributor

@f-loris f-loris commented Apr 1, 2026

Improve runway parsing to properly handle cases when data is only partially provided.

  • This change ensures that cases like R88/29//// work.
  • Even cases like R16/////// should ideally not lead to an error of the parsing and simply just provide the info that no values were reported

Thanks for taking a look.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

Please check on sonarcloud https://sonarcloud.io/project/pull_requests_list?id=io.github.mivek%3AmetarParser that the PR does not add any issue.

@f-loris f-loris changed the title improve runway parsing fix: improve runway parsing Apr 1, 2026
@mivek
Copy link
Copy Markdown
Owner

mivek commented Apr 6, 2026

Thanks for the PR @f-loris

Everything looks good, can you just edit the commit message so it follows conventional commits guidelines?

@f-loris f-loris force-pushed the improvement/runway-parsing branch from 1a534e2 to 9856a56 Compare April 7, 2026 08:37
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Apr 7, 2026

@f-loris
Copy link
Copy Markdown
Contributor Author

f-loris commented Apr 7, 2026

@mivek Thanks. I just updated the message.

@mivek mivek merged commit a9f7c94 into mivek:main Apr 8, 2026
11 checks passed
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 8, 2026

🎉 This PR is included in version 2.20.13 🎉

The release is available on:

Your semantic-release bot 📦🚀

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.

2 participants