Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix(hive_generator): Apply analyzer 5.0.0 or higher #1085

Merged
merged 4 commits into from
Oct 13, 2022
Merged

fix(hive_generator): Apply analyzer 5.0.0 or higher #1085

merged 4 commits into from
Oct 13, 2022

Conversation

jukqaz
Copy link
Contributor

@jukqaz jukqaz commented Oct 4, 2022

Update analyzer 5.0.0 or higher

@jukqaz jukqaz changed the title fix: Apply analyzer 5.0.0 or higher fix(hive_generator): Apply analyzer 5.0.0 or higher Oct 4, 2022
@aykuttasil
Copy link

Please merge the PR!

@pro100andrey
Copy link

Hi @jukaz, I have issue with enum generation,

type 'EnumElementImpl' is not a subtype of type 'ClassElement' in type cast

code example:

import 'package:hive/hive.dart';

import 'hive_object_types.dart';

part 'project_icon.g.dart';

@HiveType(typeId: HiveObjectTypes.icon)
enum ProjectIcon {
  @HiveField(0)
  heard,

  @HiveField(1)
  mask,
}

thanks

@pro100andrey
Copy link

@jukqaz Sorry, until I was writing you have updated it )

@chuiizeet
Copy link

Currently i use this PR, please merge!

Copy link
Contributor

@themisir themisir left a comment

Choose a reason for hiding this comment

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

LGTM

.github/workflows/test.yml Outdated Show resolved Hide resolved
hive_generator/pubspec.yaml Show resolved Hide resolved
jukqaz and others added 2 commits October 13, 2022 16:56
@themisir themisir merged commit c594fd3 into isar:master Oct 13, 2022
@themisir
Copy link
Contributor

Thank you for your contribution, @jukqaz 😊 I'll do a few more checks and publish a new version to pub.dev within 24 hours.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants