-
-
Notifications
You must be signed in to change notification settings - Fork 189
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
Validation of examples #235
Comments
@greyli most of the feedback is regarding |
This validation can also be done locally in Firefox with the development tools. Open with Ctrl+Shift+C and choose the tab HTML validator. |
This can be run from the command line with
Is it okay to put this in a script that is part of the repo? @greyli PS Currently there is a problem with https://bootstrap-flask-example.azurewebsites.net/icon |
Why do we need to validate the HTML of the example application? For the 500 error, I prefer to add unit test to the examples. |
It gives users a nicer starting point and might expose errors in our macros. |
I see. You could add a GitHub Actions workflow to run the scripts above when changes are made to the examples folder. Be sure to update the URL to the local URL. |
Where can we add the following links for validation of the examples?
Bootstrap 4:
Bootstrap 5:
PS There is one issue to fix with form and nav.
PPS icon overview links need to be added aftern ext release
The text was updated successfully, but these errors were encountered: