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

Added newtype support for primitive type enum #76

Merged
merged 16 commits into from
Nov 21, 2023

Conversation

Daquiver1
Copy link
Contributor

No description provided.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Nov 8, 2023
projects/jdwp/defs/schema.py Outdated Show resolved Hide resolved
projects/jdwp/codegen/new_type_generator.py Outdated Show resolved Hide resolved
projects/jdwp/codegen/new_type_generator.py Outdated Show resolved Hide resolved
projects/jdwp/codegen/new_type_generator.py Show resolved Hide resolved
projects/jdwp/codegen/new_type_generator.py Outdated Show resolved Hide resolved
projects/jdwp/codegen/new_type_generator.py Show resolved Hide resolved
projects/jdwp/codegen/new_type_generator.py Outdated Show resolved Hide resolved
Copy link
Contributor

@michalgr michalgr left a comment

Choose a reason for hiding this comment

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

Could you add 3 copyrights and rebase/merge current main ? Rebase will fix the test failure.

projects/jdwp/codegen/new_type_generator.py Outdated Show resolved Hide resolved
projects/jdwp/tests/test_primitve_type.py Show resolved Hide resolved
@michalgr michalgr merged commit 16396ad into facebookexperimental:main Nov 21, 2023
10 checks passed
Daquiver1 added a commit to Daquiver1/ExtendedAndroidTools that referenced this pull request Nov 21, 2023
* Added newtype support for primitive type enum

* Make newgen generator automatic

* Changed type of jdwp_type and changed default type to int

* Wrote test to validate primitive type mapping

* Added test for type alias definition

* Factored out function defining type alias.

* Used absolute name format for newtype

* Changed testing framework to unittest

* Added dependency for jdwp project

* Added int and string type

* Changed the expected start

* Updated test and removed int and byte from primitive types.

* Removed int and boolean type from new type generator

* doc: add copyright
Daquiver1 added a commit to Daquiver1/ExtendedAndroidTools that referenced this pull request Nov 23, 2023
* Added newtype support for primitive type enum

* Make newgen generator automatic

* Changed type of jdwp_type and changed default type to int

* Wrote test to validate primitive type mapping

* Added test for type alias definition

* Factored out function defining type alias.

* Used absolute name format for newtype

* Changed testing framework to unittest

* Added dependency for jdwp project

* Added int and string type

* Changed the expected start

* Updated test and removed int and byte from primitive types.

* Removed int and boolean type from new type generator

* doc: add copyright
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants