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

Add fix Blueliv collector requirements to CHANGELOG.md #2173

Merged
merged 1 commit into from
May 6, 2022

Conversation

gutsohnCERT
Copy link
Contributor

fix #2161

Please read the notes below and replace them with the description of you pull request

Make sure you follow the instructions in the Developer Guide - it describes how to run the test suite and which coding rules to follow.

Commits

Please review your commits before creating a pull request. We try to keep our commit history clean.
If you had to fix something in your code and added another commit on top of your existing work, please squash your commits.
This makes it a lot easier to find out why things work the way they do later on.

But please also keep in mind that one commit should only contain changes that belong together.

The cardinal rule for creating good commits is to ensure there is only one "logical change" per commit

Thus, if your changes affect multiple files and also contains tests and documentation (which it should), it might make sense to have the tests and the documentation in a separate commit.

Commit Messages

Please explain your changes also in the git commit message.
The commit message should contain a subject that gives an overview of the changes in the commit and is limited to 72 characters.
It should start with a capital letter and it should not end with a period.
Below the subject should be, separated by an empty line, the body of the commit message. The body should explain what the commit changes and why it changes thing the way it does.
Explain your modification and also explain why you didn't chose a different approach.
See also How To Write a Git Commit Message.

Description

If you are following the commit message guidelines above, all the relevant information should already be part of the commit message.
If there is anything else you want to add, feel free to do this here.

@gethvi
Copy link
Contributor

gethvi commented May 5, 2022

Thank you!

@sebix sebix added this to the 3.1.0 milestone May 6, 2022
@sebix sebix added bug Indicates an unexpected problem or unintended behavior documentation Indicates a need for improvements or additions to documentation labels May 6, 2022
@codecov-commenter
Copy link

codecov-commenter commented May 6, 2022

Codecov Report

Merging #2173 (92f62d6) into develop (6e1c1bf) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #2173   +/-   ##
========================================
  Coverage    76.34%   76.34%           
========================================
  Files          441      441           
  Lines        23649    23649           
  Branches      3739     3739           
========================================
  Hits         18055    18055           
  Misses        4857     4857           
  Partials       737      737           

Copy link
Member

@sebix sebix left a comment

Choose a reason for hiding this comment

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

Thanks!

@sebix sebix merged commit 0221a25 into certtools:develop May 6, 2022
@sebix sebix self-assigned this May 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior documentation Indicates a need for improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants