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

Add --profile build argument #192

Merged
merged 8 commits into from
Dec 19, 2024
Merged

Add --profile build argument #192

merged 8 commits into from
Dec 19, 2024

Conversation

Johennes
Copy link
Collaborator

Fixes: #190

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 You're just so quick! Thank you so much.

I've approved this with nits. I don't think they're big enough to wait for another review cycle, but I would like these fixed before landing.

Also: would you mind adding an entry to the CHANGELOG.md? I think this qualifies as "✨ What's New"

crates/ubrn_cli/src/ios.rs Outdated Show resolved Hide resolved
crates/ubrn_cli/src/android.rs Outdated Show resolved Hide resolved
crates/ubrn_cli/src/android.rs Outdated Show resolved Hide resolved
xtask/src/run/rust_crate.rs Outdated Show resolved Hide resolved
@Johennes
Copy link
Collaborator Author

Also: would you mind adding an entry to the CHANGELOG.md? I think this qualifies as "✨ What's New"

Oh, I completely forgot that we have a changelog now. 🤦‍♂️

@Johennes Johennes merged commit 1374831 into main Dec 19, 2024
18 checks passed
@Johennes Johennes deleted the johannes/profile branch December 19, 2024 13:32
@jhugman jhugman mentioned this pull request Jan 30, 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.

Allow specifying the profile for cargo build
2 participants