Skip to content

Conversation

@Frzk
Copy link

@Frzk Frzk commented Apr 9, 2025

Fix #104

schneems and others added 8 commits March 27, 2025 13:24
* Ensure workflow uses same bundler version

* Fix test
Signed-off-by: Pablo Temporini <ptemporini@salesforce.com>
* Add Heroku YAML report builder

This utility class writes a yaml file for capturing observability metrics.

* Add Ruby version metrics to global report

* Implement bin/report

* Report bundler version

* Observe railties and rack versions

* Changelog

* Document error tolerance behavior

* Update captured ruby version information

* Fix tests

* Document RubyVersion accessors

* Cleaner major ruby version

* Remove unused legacy ruby logic

We no longer provide Ruby 1.9.2

* Silence warnings
Co-authored-by: heroku-linguist[bot] <136119646+heroku-linguist[bot]@users.noreply.github.com>
The `bin/report` structure is flat. All keys must map to values that are un-nested. Ruby's `to_yaml` will happily serialize ruby objects:

```
irb(main):004> puts Gem::Version.new("3.4.2").to_yaml
--- !ruby/object:Gem::Version
version: 3.4.2
```

This commit fixes this problem by inspecting if the value being serialized is an object or not. If it is, it is then converted into a string.
Co-authored-by: heroku-linguist[bot] <136119646+heroku-linguist[bot]@users.noreply.github.com>
@Frzk Frzk self-assigned this Apr 9, 2025
@Frzk Frzk requested a review from EtienneM April 9, 2025 09:38
@Frzk Frzk enabled auto-merge April 9, 2025 09:38
Comment on lines +6 to +7
## [v299] - 2025-04-07

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

haha interesting changelog :D

@Frzk Frzk merged commit 6be9e71 into master Apr 9, 2025
1 check passed
@Frzk Frzk deleted the deps/upstream_v299 branch April 9, 2025 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Sync with upstream from v297 to v299

5 participants