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

Nmc/generalsettings manage component visibility local #51

Conversation

AnkurBavisa
Copy link

temporary create PR of upstream PR

@AnkurBavisa AnkurBavisa added custom MagentaCLOUD customisation approved labels Jul 5, 2023
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ clang-tidy found issue(s) with the introduced code (1/1)

@@ -60,6 +60,7 @@ class OWNCLOUDSYNC_EXPORT ConfigFile
[[nodiscard]] bool showConfigBackupWarning() const;

bool exists();
bool isWidgetVisible() const;
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ modernize-use-nodiscard ⚠️
function isWidgetVisible should be marked [[nodiscard]]

Suggested change
bool isWidgetVisible() const;
[[nodiscard]] bool isWidgetVisible() const;

@codecov-commenter
Copy link

Codecov Report

Merging #51 (9cf3d91) into master (953dee3) will decrease coverage by 0.30%.
The diff coverage is 25.00%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master      #51      +/-   ##
==========================================
- Coverage   60.36%   60.07%   -0.30%     
==========================================
  Files         143      146       +3     
  Lines       18518    18719     +201     
==========================================
+ Hits        11179    11246      +67     
- Misses       7339     7473     +134     
Impacted Files Coverage Δ
src/libsync/configfile.cpp 26.19% <0.00%> (-0.13%) ⬇️
src/libsync/configfile.h 100.00% <100.00%> (ø)

... and 29 files with indirect coverage changes

@AnkurBavisa AnkurBavisa force-pushed the nmc/generalsettings_manage_component_visibility_local branch from 9cf3d91 to fdd508f Compare July 10, 2023 08:31
Signed-off-by: Ankur Bavisa <Ankur.Bavisa@t-systems.com>
@AnkurBavisa AnkurBavisa force-pushed the nmc/generalsettings_manage_component_visibility_local branch from fdd508f to 0bde8d7 Compare July 11, 2023 09:51
@tsdicloud tsdicloud added ziteration25 This doesn't seem right and removed custom MagentaCLOUD customisation labels Aug 15, 2023
@tsdicloud tsdicloud closed this Oct 31, 2023
@memurats memurats deleted the nmc/generalsettings_manage_component_visibility_local branch September 30, 2024 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ziteration25 This doesn't seem right
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants