Skip to content
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

Split compat job by platform and version #211

Merged
merged 7 commits into from
Jan 28, 2025
Merged

Conversation

Johennes
Copy link
Collaborator

This is an attempt at splitting the "Compat" job so that we can report on it more granularly instead of having a single "Compat | failing" badge in the README.

@jhugman jhugman self-requested a review January 27, 2025 14:09
Copy link
Owner

@jhugman jhugman left a comment

Choose a reason for hiding this comment

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

Oh yes, I very much like this. This will mean we can put more badges on the README.

Thank you!

@Johennes
Copy link
Collaborator Author

Oh yes, I very much like this. This will mean we can put more badges on the README.

Yes, exactly. It does require some duplication but it seemed better than having one version make the single badge red.

Also, splitting the historic versions (the ones we consciously choose to support) from the latest versions (which can break any time) seemed sensible.

@Johennes
Copy link
Collaborator Author

The current changes will look like this in the README:

Screenshot 2025-01-27 at 15 36 14

@Johennes Johennes marked this pull request as ready for review January 27, 2025 14:57
Copy link
Owner

@jhugman jhugman left a comment

Choose a reason for hiding this comment

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

👨‍🍳 🤌 🚀 🚢

@Johennes Johennes merged commit 082a4f9 into main Jan 28, 2025
13 of 14 checks passed
@Johennes Johennes deleted the johannes/split-compat-job branch January 28, 2025 06:11
This was referenced Jan 28, 2025
jhugman added a commit that referenced this pull request Jan 30, 2025
It's time for a release. 

If your change hasn't landed before this release, don't worry, releases
are cheap! We can do another one!

# 0.28.3-2
## ✨ What's New
* Add `--profile` build argument
([#192](#192))
  * Thank you [@Johennes](https://github.com/Johennes)!

## 🦊 What's Changed

* Adjust template to allow for hot reload via metro of running apps
([#207](#207)).
* Stabilise `require.resolve` by looking up `package.json` instead of
entrypoint
([#200](#200)).
  * Thank you [@hassankhan](https://github.com/hassankhan)!
* Split compat job by platform and version
([#211](#211)).
* This shows on the README.md if builder-bob or React Native has changed
breaking the tutorial.
  * Thank you [@Johennes](https://github.com/Johennes)!
* Fixed GC'ing objects with callbacks intermittent crasher
([#208](#208)
and
[#209](#209))
* Reproducibly pick the same library file when using `--and-generate`
([#194](#194))
  * Thank you [@Johennes](https://github.com/Johennes)!

## 🌏🕸️ WASM!
* Fixtures `coverall`, `custom-types-example`, `enum-types`,
`trait-methods`
([#202](#202)).
* Switched from passing `ArrayBuffer`s to using `Uint8Array`, to
accommodate WASM better.
([#187](#187))
Callbacks now have UniffiResult to communicate between typescript and
C++
([#205](#205)).
* Fixtures `coverall2` and `rondpoint`
([#191](#191)).
* Fixture `arithmetic`
([#188](#188)).

## 📰 Documentation
* Remove duplicate parentheses
([#203](#203)).
* Minor typo fixes in GC docs
([#204](#204)).
* Remove reference to name field in the ubrn.config.yaml docs
([#189](#189)).

**Full Changelog**:
0.28.3-1...0.28.3-2
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.

2 participants