Skip to content

Conversation

@egekorkan
Copy link
Contributor

@egekorkan egekorkan commented Jul 11, 2025

Description of Changes

Just a simple oversight in an example mentioned in #2038

Related Issue

Closes #2038

Type of Change


Preview | Diff

@egekorkan egekorkan added the Editorial Issues with no technical impact on implementations label Jul 11, 2025
@egekorkan
Copy link
Contributor Author

Reviewed in the TD call, merging

@egekorkan egekorkan merged commit 814fb21 into main Jul 16, 2025
2 checks passed
@egekorkan egekorkan deleted the ege-fix-2038 branch July 16, 2025 14:34
@TallTed
Copy link
Member

TallTed commented Jul 16, 2025

It seems rather odd for a "simple oversight in an example" to change ~5000 lines. This might want another look.

@egekorkan
Copy link
Contributor Author

It seems rather odd for a "simple oversight in an example" to change ~5000 lines. This might want another look.

That is true but index.html is always auto-generated. All the changes seem like formatting changes though. I will have a deeper look.

@egekorkan
Copy link
Contributor Author

It seems I forgot to run format and just did with b0fbec2 . Not sure why the gh action did not show a failure...

egekorkan added a commit that referenced this pull request Jul 17, 2025
@TallTed
Copy link
Member

TallTed commented Jul 17, 2025

index.html is always auto-generated

Ah. Bitten by artefacts being kept in source management. Best to avoid this, wherever and whenever possible.

egekorkan added a commit that referenced this pull request Aug 7, 2025
* start reorganizing the intro for bindings

* Apply suggestions from code review

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>

* change to td 2.0 and fix broken links (#2113)

* change to td 2.0 and fix broken links

* use version correctly

* run formatting

* fix: remove _value suffix from enum (#2117)

* chore: run format

* start reorganizing the intro for bindings

* chore: run format

* add prettier to rendering script

* finish the basic intro for bindings

* fix conflicts

* rework other binding sections

* adapt platform binding text and remove respec errors

* Apply suggestions from code review

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>

* fix english mistakes

* Apply suggestions from code review

Co-authored-by: danielpeintner <daniel.peintner@gmail.com>

* Apply suggestions from code review

Co-authored-by: danielpeintner <daniel.peintner@gmail.com>

* fix linking mistakes

---------

Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
Co-authored-by: danielpeintner <daniel.peintner@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Editorial Issues with no technical impact on implementations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unexplained value transformation or inconsistency: metric_value and metric

3 participants