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

Refactor properties, connector injections and initializing beans #276

Merged
merged 7 commits into from
Jan 7, 2024

Conversation

dabico
Copy link
Member

@dabico dabico commented Jan 7, 2024

Most notable changes:

  • Moved the @Connector annotation processor to another package
  • Properties of @Connector components are injected directly with @Value
  • The properties for @Connector components reside outside the analysis and cleaner properties
  • Some package-private beans have been removed
  • LanguageService and GitConnector both implement InitializingBean now, and the two existing ones have been removed

These changes also impact the application.properties. The README has been updated to reflect this.

@dabico dabico merged commit 7a26934 into master Jan 7, 2024
6 checks passed
@dabico dabico deleted the refactoring/beans branch January 7, 2024 18:54
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.

1 participant