Switch current version to documented version - #13918
Conversation
sfshaza2
left a comment
sfshaza2
left a comment
There was a problem hiding this comment.
Thankyou! I do prefer when it sticks to major.minor.
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request replaces the 'currentFlutterVersion' configuration with 'documentedFlutterVersion' (in 'major.minor' format) across the documentation and layout components, adding validation to ensure the version is correctly formatted. The feedback suggests improving the validation error message to explicitly guide contributors to use a quoted string, preventing issues where YAML parsers interpret unquoted versions as doubles.
|
Staged preview of the updated docs.flutter.dev site (updated for commit ab0e656): https://flutter-docs-prod--docs-pr13918-misc-simplified-docume-j464iir7.web.app |
|
Staged preview of the updated flutter.dev site (updated for commit bb0d8c4): https://flutter-dev-230821--www-pr13918-misc-simplified-docume-o5k7okli.web.app |
3 participants
Keep the site variable focused by only specifying the
major.minorversion as patches should generally not affect the docs at a grand scale and we might not update the version every patch. Also update the name fromcurrentFlutterVersiontodocumentedFlutterVersionto better reflect its purpose and usage.