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

remove the unused [comment] section from config file (bugfix) #1158

Conversation

bladernr
Copy link
Collaborator

@bladernr bladernr commented Apr 8, 2024

... as it is apparently not used and has been forgotten about for quite some time. #1149

Description

Just removes the old unused [comment] section from the config file for the server cert tools.

Resolved issues

#1149

Documentation

Deletion of something everyone had forgotten, no docs changes needed.

Tests

Removed the section, ran a launcher, saw no warnings about the section.

…parently not used and has been forgotten about for quite some time. #1149
@bladernr bladernr changed the title Fix: remove the unused [comment] section from config file as it is ap… Fix: remove the unused [comment] section from config file Apr 8, 2024
Copy link

codecov bot commented Apr 8, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 43.09%. Comparing base (be41d70) to head (8f34e09).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1158   +/-   ##
=======================================
  Coverage   43.09%   43.09%           
=======================================
  Files         355      355           
  Lines       38602    38602           
  Branches     6556     6556           
=======================================
  Hits        16634    16634           
  Misses      21302    21302           
  Partials      666      666           
Flag Coverage Δ
provider-certification-server 57.14% <ø> (ø)

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

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

@bladernr bladernr changed the title Fix: remove the unused [comment] section from config file bugfix: remove the unused [comment] section from config file as it is ap… Apr 8, 2024
@bladernr bladernr changed the title bugfix: remove the unused [comment] section from config file as it is ap… remove the unused [comment] section from config file (bugfix) Apr 8, 2024
Copy link
Collaborator

@Hook25 Hook25 left a comment

Choose a reason for hiding this comment

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

+1, ty

@Hook25 Hook25 merged commit 9ff53ee into main Apr 29, 2024
12 of 16 checks passed
@Hook25 Hook25 deleted the 1149-checkbox-complains-about-the-common-section-in-canonical-certificationconf branch April 29, 2024 10:01
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.

checkbox complains about the [common] section in canonical-certification.conf
2 participants