Skip to content

Conversation

tmsteen
Copy link

@tmsteen tmsteen commented Sep 23, 2025

Updated the metar app to utilize the new Data API from Aviation Weather. The old API was deprecated and fully decommissioned in Sept 2025.

This update also implements the HTTP module native caching mechanism. Tested with local render.

@avalys is the original author and may want to comment/approve this PR.

@tidbyt
Copy link

tidbyt bot commented Sep 23, 2025

⚠️ The automated review process is experimental and likely has bugs. Please bear with us as we iron out the kinks and enable you to ship changes at high velocity 🚀

Next Steps

Hello! Thank you so much for your change 🤜 🤛 . There are a few things you need to do:

  • Sign the CLA if you haven't already
  • Ensure your build is green! Any problem will display a proposed solution to try out
  • Get a review, either by Tidbyt Bot or by a Tidbyt engineer

Manual Review Required

Hang tight! A Tidbyt engineer will be by shortly to review your change. Here is what they will be looking for:

Test Details
App Dir All files are in a single app directory
🟡 Modules Usage of http.star requires review
🟡 Original Author The original author (avalys) does not match the PR author (tmsteen)

@tidbyt-bot
Copy link

tidbyt-bot commented Sep 23, 2025

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@tmsteen
Copy link
Author

tmsteen commented Sep 23, 2025

I have read the CLA Document and I hereby sign the CLA

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.

2 participants