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.

Sign up for GitHub

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

Jump to bottom

Remy - v1.2.0 #1

Merged
merged 18 commits into from
Jul 24, 2022
Merged

Remy - v1.2.0 #1

merged 18 commits into from
Jul 24, 2022

Conversation

Jictyvoo
Copy link
Member

  • Replace default type resolution - Now it will not use the reflect package by default.
  • Add UseReflectionType option in Config struct
  • Improve tests coverage
    • Add test to check type resolution for elements with same type-name and package-name from another module
  • Fix GetElemKey method not being able to get the type of the interface
  • Fix an error where interface and pointer of the same type were being registered as the same type
  • Create additional "Do" methods: DoGet, DoGetGen, DoGetGenFunc
  • Refactor the Storage/Injector retrieval to return an error instead of a bool

Jictyvoo added 18 commits July 20, 2022 13:08
@Jictyvoo
🍉 perf: Remove the use of reflection as default TypeName
29d40ee 
It only will be used when the ShouldGenerify flag is true
@Jictyvoo
🎩 test: Verify if type-checks respect type package
5f756c6 
- Remove unnecessary else statement
@Jictyvoo
🕯️ refactor: Move all reflection types to a new file inside package
589c91c 
- Refactor standard_injector to use named returns
- Add missing test check
@Jictyvoo
⛵ docs: Update ROADMAP.md
68befdd
@Jictyvoo
📁 refactor: Detach type methods in 2-files separated by flag
5669adb 
Has a problem with generify duck-interface not found any method
@Jictyvoo
💀 fix: Get correct interface name when generified
95e7e5f
@Jictyvoo
⚙️ feat: Add a new option to allow user choose to use reflection
9d16294 
By default, the package will not use reflection, but this will not make to be possible to know the entire path for the file

When using reflection, the type will be unique even if it has same package and name
@Jictyvoo
🔨 test: Implement equal-package test
e51650f 
Update the test to use the struct `ReflectionOptions`
@Jictyvoo
🔬 test: Fix all tests by using ReflectionOptions structure
daae4f3
@Jictyvoo
📝 Update README.md
58e324b
@Jictyvoo
📆 docs: Update CHANGELOG.md
dd60062
@Jictyvoo
⚗️ test: Add test to check the reflection type evaluation
bc0a98f
@Jictyvoo
🎳 refactor: Return error instead of bool on _get-methods_
8fdec8f 
Create new errors
@Jictyvoo
🌴 refactor: Return error alongside with element in injector
58f4855 
To not break compatibility, the exported functions was keep as the same they are, without changing its signature
@Jictyvoo
🍇 test: Check difference between pointer and interface types
1e3b415
@Jictyvoo
🌫️ feat: Add differentiation from pointer and interface types
01fe36c
@Jictyvoo
📆 docs: Update CHANGELOG.md
aa026fe
@Jictyvoo
🚀 docs: Update ROADMAP.md
8dc6e5a
@Jictyvoo Jictyvoo added bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request labels Jul 24, 2022
@Jictyvoo Jictyvoo self-assigned this Jul 24, 2022
@Jictyvoo Jictyvoo merged commit 1af51d8 into main Jul 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@Jictyvoo Jictyvoo

Labels
bug Something isn't working documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Jictyvoo