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

fix: read only system variables for properties #813

Merged
merged 2 commits into from
Jun 11, 2024
Merged

Conversation

katerina20
Copy link
Collaborator

No description provided.

@katerina20 katerina20 requested a review from andrii-bodnar June 7, 2024 02:40
Copy link

codecov bot commented Jun 7, 2024

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 64.44%. Comparing base (31a7fe1) to head (5aabbf8).

Files Patch % Lines
.../com/crowdin/cli/properties/PropertiesBuilder.java 50.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #813      +/-   ##
============================================
- Coverage     64.51%   64.44%   -0.06%     
+ Complexity     1488     1486       -2     
============================================
  Files           223      223              
  Lines          6144     6144              
  Branches        923      923              
============================================
- Hits           3963     3959       -4     
- Misses         1670     1674       +4     
  Partials        511      511              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@andrii-bodnar andrii-bodnar linked an issue Jun 11, 2024 that may be closed by this pull request
@andrii-bodnar andrii-bodnar merged commit e091e22 into main Jun 11, 2024
8 checks passed
@andrii-bodnar andrii-bodnar deleted the fix/env-config branch June 11, 2024 13:05
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.

CLI does not parse .env files with multi line variables at all
2 participants