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

Version Catalog support #950

Merged
merged 42 commits into from
Aug 23, 2022
Merged

Version Catalog support #950

merged 42 commits into from
Aug 23, 2022

Conversation

riggaroo
Copy link
Collaborator

@riggaroo riggaroo commented Aug 18, 2022

Migrated all build files to use version catalogs.

@riggaroo riggaroo requested a review from a team as a code owner August 18, 2022 10:37
@riggaroo riggaroo requested a review from ricknout August 18, 2022 10:37
@riggaroo riggaroo requested a review from ricknout August 22, 2022 12:53
Copy link
Contributor

@ricknout ricknout left a comment

Choose a reason for hiding this comment

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

Jetchat LGTM

@@ -0,0 +1,30 @@
versionCatalogUpdate {
Copy link
Contributor

@arriolac arriolac Aug 22, 2022

Choose a reason for hiding this comment

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

Add license header to this file? Same comment for all newly introduced files. https://github.com/google/addlicense can help here

Copy link
Member

@bentrengrove bentrengrove left a comment

Choose a reason for hiding this comment

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

JetSnack LGTM!

Plus the new setup looks good 👍

Copy link
Contributor

@simona-anomis simona-anomis left a comment

Choose a reason for hiding this comment

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

Owl looks good, thanks Rigs!

Copy link
Contributor

@astamato astamato left a comment

Choose a reason for hiding this comment

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

Jetcaster LGTM
Great work Rigs!

Copy link
Contributor

@JolandaVerhoef JolandaVerhoef left a comment

Choose a reason for hiding this comment

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

Jetnews LGTM!

Copy link
Contributor

@Gurupreet Gurupreet left a comment

Choose a reason for hiding this comment

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

Reply LGTM, Thanks you 🥇

@riggaroo riggaroo merged commit 70d05ef into main Aug 23, 2022
@riggaroo riggaroo deleted the rebecca/version_catalogs_2 branch August 23, 2022 16:55
vfishv added a commit to vfishv/compose-samples that referenced this pull request Sep 25, 2022
* commit '5bc3451b11f9aae111b8a6dca0199a317c997a2b': (331 commits)
  Updating to safe flow collection with collectAsStateWithLifecycle() (android#970)
  [Jetsurvey] Converted to use Material 3 (android#954)
  [JetNews] Remove invalid comment (android#966)
  [Jetnews] Update to Material3 (android#964)
  Compose latest (android#961)
  Disable auto creation of deps PRs for renovate (android#960)
  Version Catalog support  (android#950)
  Fixing: Remove Unused If-Else condition (android#884)
  [Jetcaster] Fix recomposition loop in verticalGradientScrim (android#946)
  Fix readme image size (android#926)
  Fix a11y bugs (android#914)
  Update Androidx Lifecycle deps to v2.5.1 (android#932)
  Update plugin com.android.test to v7.2.2 (android#931)
  [Crane] Fix destinations not in list issue. (android#938)
  Update main | Compose 1.2.1 | Compiler 1.3.0 | Kotlin 1.7.10 (android#924)
  Update accompanist_version to v0.25.1
  added licence to shapes
  added shapes for material
  Added a fix to not show back button on permanent drawer, added Material shapes
  [All] Updating to Compose 1.2 (android#905)
  ...
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.

10 participants