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

Format unsaved entity file paths #1158

Merged
merged 1 commit into from
Jan 11, 2025
Merged

Conversation

gandarez
Copy link
Member

@gandarez gandarez commented Jan 9, 2025

This PR removes the validation of unsaved entity when formatting the entity filepath.

@gandarez gandarez self-assigned this Jan 9, 2025
@alanhamlett alanhamlett changed the title Prioritize windows when formatting filepath Format unsaved entity file paths Jan 10, 2025
@alanhamlett alanhamlett force-pushed the feature/format-windows branch from ca3e5ed to f75df94 Compare January 10, 2025 02:01
@alanhamlett alanhamlett marked this pull request as ready for review January 10, 2025 02:02
@alanhamlett alanhamlett force-pushed the feature/format-windows branch 2 times, most recently from 8b715ce to f100901 Compare January 10, 2025 02:04
@gandarez gandarez force-pushed the feature/format-windows branch 3 times, most recently from 0e366a3 to 69b27a6 Compare January 10, 2025 11:54
@gandarez gandarez force-pushed the feature/format-windows branch from 69b27a6 to 6f6d40e Compare January 10, 2025 12:03
@alanhamlett alanhamlett merged commit 43ab913 into develop Jan 11, 2025
20 of 21 checks passed
@alanhamlett alanhamlett deleted the feature/format-windows branch January 11, 2025 01:12
Copy link

codecov bot commented Jan 11, 2025

Codecov Report

Attention: Patch coverage is 35.71429% with 9 lines in your changes missing coverage. Please review.

Project coverage is 62.99%. Comparing base (02eefc9) to head (6f6d40e).
Report is 2 commits behind head on develop.

Files with missing lines Patch % Lines
pkg/heartbeat/format.go 30.76% 9 Missing ⚠️
@@             Coverage Diff             @@
##           develop    #1158      +/-   ##
===========================================
- Coverage    62.99%   62.99%   -0.01%     
===========================================
  Files          386      386              
  Lines        16914    16923       +9     
===========================================
+ Hits         10655    10660       +5     
- Misses        5686     5690       +4     
  Partials       573      573              
Flag Coverage Δ
unittests 62.99% <35.71%> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
pkg/windows/windows.go 80.00% <100.00%> (+1.59%) ⬆️
pkg/heartbeat/format.go 34.61% <30.76%> (-0.17%) ⬇️

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