Skip to content

Replace @Autowired with constructor injection#99

Merged
omatheusmesmo merged 2 commits intoomatheusmesmo:developfrom
Invokedzz:refactor/replace-di-with-constructor-injection
Mar 7, 2026
Merged

Replace @Autowired with constructor injection#99
omatheusmesmo merged 2 commits intoomatheusmesmo:developfrom
Invokedzz:refactor/replace-di-with-constructor-injection

Conversation

@Invokedzz
Copy link
Copy Markdown
Contributor

@Invokedzz Invokedzz commented Mar 5, 2026

Nowadays, It is recommended to use constructor injection instead of Autowired annotation; it enforces better code readability, testability, immutability, dependencies are initialized in during construction and ensures compile-time safety.

Changes

  • Replaced Autowired with constructor injection in controller/service classes.

Tests

  • Tested the changes locally;
  • Added new tests for the new features

omatheusmesmo and others added 2 commits March 1, 2026 15:37
…o#90)

* fix(ci): update branch enforcer permissions and enable Gemini reviews for main

* fix: remove redundant HttpClient injection

* fix(ci): update event type check for pull_request_target

* fix: Remove placeholders, debug logs, and implement missing logic

* fix: format e ajust never used

* docs: Update Readme

* fix(ci): fix Gemini dispatch triggers to include synchronize and more PR events

---------

Co-authored-by: Matheus André <92062874+matheusandre1@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 5, 2026

🤖 Hi @Invokedzz, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 5, 2026

🤖 I'm sorry @Invokedzz, but I was unable to process your request. Please see the logs for more details.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 5, 2026

🤖 Hi @Invokedzz, I've received your request, and I'm working on it now! You can track my progress in the logs for more details.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 5, 2026

🤖 I'm sorry @Invokedzz, but I was unable to process your request. Please see the logs for more details.

@omatheusmesmo omatheusmesmo merged commit 0013e7e into omatheusmesmo:develop Mar 7, 2026
22 of 24 checks passed
@omatheusmesmo
Copy link
Copy Markdown
Owner

Great job, @Invokedzz !

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