[test_operator] Fix sync README script#3855
[test_operator] Fix sync README script#3855kstrenkova wants to merge 1 commit intoopenstack-k8s-operators:mainfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
kstrenkova
force-pushed
the
fix-sync-readme-script
branch
from
584701b to
78b3cf4
Compare
kstrenkova
changed the title
|
Hello @michburk 👋 I have created a change that removes the line numbers from the link and hopefully also removes all the dead code created by such change. Additionally, with my new parameter I noticed that even if the default is an empty array |
|
Looks good @kstrenkova thank you! Before we merge this, do you think we should merge #3698, then rebase this on top + re-run the updated sync script in this pr? 3689 has been open for a good while and I don't want to delay it further with another snag because of more readme sync changes 😅 |
Empty lists/dicts now display inline (e.g., []) instead of redirecting to defaults file. Removed line number tracking and anchors - links now point to main.yml file without #L123 fragments for simpler navigation. This change was also done, so that when adding a new parameter, users don't have to run the sync script first to apply changes to the line numbers. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com> Signed-off-by: Katarina Strenkova <kstrenko@redhat.com>
kstrenkova
force-pushed
the
fix-sync-readme-script
branch
from
78b3cf4 to
51fffb3
Compare
|
@michburk I approved the other MR in the morning, rebased this one atop it and I think this one is now ready to get merged :D |
2 participants
Empty lists/dicts now display inline (e.g., []) instead of redirecting to defaults file. Removed line number tracking and anchors - links now point to main.yml file without #L123 fragments for simpler navigation. This change was also done, so that when adding a new parameter, users don't have to run the sync script first to apply changes to the line numbers.