Skip to content

Conversation

vaslabs
Copy link
Contributor

@vaslabs vaslabs commented Sep 18, 2025

For #5886

This PR allows for viewBinding to work via the databinding-compiler.

The option is enabled similarly with Gradle and Mill's compose flags.

It's implementation is adapted from the CLI tool here

Co-authored by @souvlakias

Jetchat example

image

vaslabs added a commit to vaslabs-ltd/compose-samples-with-mill that referenced this pull request Sep 18, 2025
vaslabs added a commit to vaslabs-ltd/compose-samples-with-mill that referenced this pull request Sep 18, 2025
Copy link
Contributor

autofix-ci bot commented Sep 18, 2025

Hi! I'm autofix logoautofix.ci, a bot that automatically fixes trivial issues such as code formatting in pull requests.

I would like to apply some automated changes to this pull request, but it looks like I don't have the necessary permissions to do so. To get this pull request into a mergeable state, please do one of the following two things:

  1. Allow edits by maintainers for your pull request, and then re-trigger CI (for example by pushing a new commit).
  2. Manually fix the issues identified for your pull request (see the GitHub Actions output for details on what I would like to change).

@vaslabs vaslabs changed the title databinding compiler scaffolding Android: databinding compiler scaffolding Sep 19, 2025
@vaslabs vaslabs marked this pull request as ready for review September 23, 2025 10:39
@vaslabs
Copy link
Contributor Author

vaslabs commented Sep 23, 2025

good job @souvlakias , thanks for the implementation.

should be ready for review

@vaslabs vaslabs changed the title Android: databinding compiler scaffolding Android: databinding compiler with view binding example Sep 23, 2025
@vaslabs
Copy link
Contributor Author

vaslabs commented Sep 24, 2025

@souvlakias almost forgot, can you annotate the new classes with experimental

@vaslabs vaslabs requested a review from lefou September 25, 2025 08:57
@vaslabs
Copy link
Contributor Author

vaslabs commented Sep 26, 2025

retested after the changes

image

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.

3 participants