Skip to content

Conversation

@RishikeshChirammelAjit
Copy link
Contributor

Hello hello :)

  • maintainer.json to get standard-agent featured on forklore.in
  • fixing get-featured.md : the parse_issue.py mentioned does not exist.

@idlip
Copy link
Collaborator

idlip commented Oct 15, 2025

Hi!

Thanks for your interest in forklore!

Just curious, how did you come across Forklore.in or were introduced?

@RishikeshChirammelAjit
Copy link
Contributor Author

Hi!

Thanks for your interest in forklore!

Just curious, how did you come across Forklore.in or were introduced?

I was directed your way by Ansh Arora

@idlip
Copy link
Collaborator

idlip commented Oct 16, 2025

Yep, glad to know that.

The page renders as expected.

Just two nitpick:

  1. You missed answering 3 questions, if possible can you please fill-in your first thoughts?

  2. We use cspell (node pkg) to avoid some spelling mistakes (although redundant tbh), can you please also add those words to .cspell/ on any file there?
    If you are unsure on (2) its alright, I can merge and fix it later

With that it'd be ready to merge.
Thank you!

@RishikeshChirammelAjit
Copy link
Contributor Author

Yep, glad to know that.

The page renders as expected.

Just two nitpick:

  1. You missed answering 3 questions, if possible can you please fill-in your first thoughts?
  2. We use cspell (node pkg) to avoid some spelling mistakes (although redundant tbh), can you please also add those words to .cspell/ on any file there?
    If you are unsure on (2) its alright, I can merge and fix it later

With that it'd be ready to merge. Thank you!

Sure thing. Nitpicks are great.

  1. I filled in for 1 of the questions, but tbh I do not have a song and an emoji 😅, if I have to add that in, it wont be an honest one.

  2. I checked previous PRs and I have tried fixing the issues using cspell. Please do let me know if I have missed anything

@idlip
Copy link
Collaborator

idlip commented Oct 16, 2025

Thanks for quick update.

Looks good to me. Ready to merge.

I filled in for 1 of the questions, but tbh I do not have a song and an emoji 😅, if I have to add that in, it wont be an honest one.

Sure thing no worries., but if you had chosen some Emoji, this page could have benefit to show in grid: #108 😉

@RishikeshChirammelAjit
Copy link
Contributor Author

Thanks for quick update.

Looks good to me. Ready to merge.

I filled in for 1 of the questions, but tbh I do not have a song and an emoji 😅, if I have to add that in, it wont be an honest one.

Sure thing no worries., but if you had chosen some Emoji, this page could have benefit to show in grid: #108 😉

That is fair. I have added in the emoji now. Cheers :)

@idlip idlip merged commit da8a7ee into fossunited:develop Oct 21, 2025
1 check 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.

2 participants