Skip to content

Add multiple spec links feature #4901

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

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

suprith-hub
Copy link

@suprith-hub suprith-hub commented Mar 27, 2025

Hi @jrobbins @KyleJu can I get a review on this PR,
I've added support for multiple spec_links in PR for #4504.

I need help with:

  1. Understanding processes.py and legacy_converters.py — I have replaced spec_link with spec_links, I would need to know the impact and function of these files...
  2. Writing/visualizing unit tests — I was able to run the tests but not able to figure out the failed tests, I need assistance in it.
  3. Migration script for mapping existing spec_link to array of spec_links

And for old features page I have added the hoverable drop down which was an anchor with maturity text
Thanks!

@suprith-hub suprith-hub changed the title Add multiple spec link feature Add multiple spec links feature Apr 1, 2025
@suprith-hub
Copy link
Author

@jrobbins Could I get a quick go-ahead on the draft PR? I’d like to ensure I’m on the right track before finalizing the remaining changes for this PR.

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.

1 participant