Skip to content

Allow to use UtUtils for different languages in one project #1296

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

Merged
merged 3 commits into from
Nov 2, 2022

Conversation

EgorkaKulikov
Copy link
Collaborator

Description

We use base package name for all UtUtils classes.
After that, we create language-specific package in it named after the language and locate the class in it.
It allows to avoid name clashing

This change introduces different util classes for different languages, so some objects are replaced with classes, some methods are refactored to know about the current language.

Fixes # (1080)

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • Refactoring (typos and non-functional changes)

How Has This Been Tested?

Manual Scenario

Check that there are no more compilation clashes.
Check that the logic of util classes and their usage did not changed.

Checklist (remove irrelevant options):

This is the author self-check list

  • The change followed the style guidelines of the UTBot project
  • Self-review of the code is passed
  • The change contains enough commentaries, particularly in hard-to-understand areas
  • New documentation is provided or existed one is altered
  • No new warnings
  • New tests have been added
  • All tests pass locally with my changes

@EgorkaKulikov EgorkaKulikov enabled auto-merge (squash) November 2, 2022 15:32
@EgorkaKulikov EgorkaKulikov merged commit bc8b627 into main Nov 2, 2022
@EgorkaKulikov EgorkaKulikov deleted the egor/ututils_packages branch November 2, 2022 15:57
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.

2 participants