community icon indicating copy to clipboard operation
community copied to clipboard

Removed API key and tidied up display

Open vipulchhajer opened this issue 4 months ago • 1 comments

Description

Tidied up display

Copilot

copilot:all

vipulchhajer avatar Feb 26 '24 01:02 vipulchhajer

⚠️ 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 File 'apps/indianews/.idea/.gitignore' is in apps/indianews/.idea
🟡 Modules Usage of http.star requires review
Original Author The original author matches the PR author

Previews

apps/indianews/india_news.star:

tidbyt[bot] avatar Feb 26 '24 17:02 tidbyt[bot]

Hi @matslina Would you mind giving this is quick review?

vipulchhajer avatar Mar 10 '24 23:03 vipulchhajer

Hey @vipulchhajer. This PR adds some files under .idea that I'd rather not merge into the repo. But the starlark changes are fine, happy to merge those!

matslina avatar Mar 18 '24 15:03 matslina

Hi @matslina Thanks. The files under .idea seem like a mistake on my end. Yes, please merge the starlark changes. Let me know if I need to do anything.

vipulchhajer avatar Mar 21 '24 22:03 vipulchhajer

~~hi @matslina Bumping this :) Mind merging in the starlark changes? Or let me know if there's something I need to do 🙏 ~~

Edit: Silly me for assuming I didn't have to update this PR. Bear me with me, I'm learning to code. Will put up pr modifications soon.

vipulchhajer avatar Apr 06 '24 17:04 vipulchhajer