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

Update Changelog #240

Merged
merged 8 commits into from
May 29, 2022
Merged

Update Changelog #240

merged 8 commits into from
May 29, 2022

Conversation

danielfeismann
Copy link
Member

@danielfeismann danielfeismann commented May 20, 2022

resolves #221

Depends on #244

@danielfeismann danielfeismann added the documentation Improvements or additions to documentation label May 20, 2022
@danielfeismann danielfeismann self-assigned this May 20, 2022
@danielfeismann danielfeismann marked this pull request as ready for review May 20, 2022 13:10
@danielfeismann
Copy link
Member Author

Unsure also to add changes in documentation. What do you thing?

@sonarqubegithubprchecks

This comment has been minimized.

@codecov
Copy link

codecov bot commented May 20, 2022

Codecov Report

Merging #240 (91c12ea) into dev (5c9d993) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev     #240   +/-   ##
=======================================
  Coverage   79.63%   79.63%           
=======================================
  Files         156      156           
  Lines        5751     5751           
  Branches       79       79           
=======================================
  Hits         4580     4580           
  Misses       1171     1171           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5c9d993...91c12ea. Read the comment docs.

Copy link
Member

@sebastian-peter sebastian-peter left a comment

Choose a reason for hiding this comment

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

Looks great, thanks for the work. I just have some minor comments, and at least one issue is missing: #220

CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
CHANGELOG.md Outdated Show resolved Hide resolved
@sonarqubegithubprchecks

This comment has been minimized.

@sonarqubegithubprchecks

This comment has been minimized.

CHANGELOG.md Outdated Show resolved Hide resolved
@danielfeismann
Copy link
Member Author

!test

Copy link
Member

@sebastian-peter sebastian-peter left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@sonarqubegithubprchecks

This comment has been minimized.

Copy link
Contributor

@t-ober t-ober left a comment

Choose a reason for hiding this comment

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

Thanks for tackling. Found some more 🙃

CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
CHANGELOG.md Show resolved Hide resolved
@sonarqubegithubprchecks

This comment has been minimized.

@sebastian-peter sebastian-peter requested review from t-ober and removed request for sebastian-peter May 24, 2022 13:32
Copy link
Member

@staudtMarius staudtMarius left a comment

Choose a reason for hiding this comment

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

Looks great. I found no problems with the issues I worked on.

@danielfeismann
Copy link
Member Author

!test

@sonarqubegithubprchecks
Copy link

Passed

Analysis Details

0 Issues

  • Bug0 Bugs
  • Vulnerability0 Vulnerabilities
  • Code Smell0 Code Smells

Coverage and Duplications

  • No coverage informationNo coverage information (81.60% Estimated after merge)
  • No duplication informationNo duplication information (0.00% Estimated after merge)

Project ID: edu.ie3:simona

View in SonarQube

@sebastian-peter sebastian-peter merged commit c2418ef into dev May 29, 2022
@sebastian-peter sebastian-peter deleted the df/#221-update-changelog branch May 29, 2022 19:20
@sebastian-peter sebastian-peter added this to the Version 3.0 milestone Feb 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update CHANGELOG.md
4 participants