-
Notifications
You must be signed in to change notification settings - Fork 160
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
[DX-908]Generate url availability file #3986
Conversation
PR Analysis
PR Feedback💡 General suggestions: The PR is generally well-structured, but it lacks error handling in some places. It would be beneficial to add more comments to explain the purpose of certain blocks of code, especially those involving complex logic. Also, consider adding unit tests to ensure the correctness of the code. 🤖 Code feedback:
✨ Usage guide:Overview:
With a configuration file, use the following template:
See the review usage page for a comprehensive guide on using this tool. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
PR LGTM and approving. As discussed in meet add comments for block of codes that stores similar and diff paths.
✅ PS. Pls add /docs/nightly to the end of url
To edit notification comments on pull requests, go to your Netlify site configuration. |
✅ PS. Pls add /docs/nightly to the end of url
To edit notification comments on pull requests, go to your Netlify site configuration. |
This pr contains the github action that will be run to produce the json file that will be used to create the dynamic version selector.