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 version: 1.2.12 -> 1.2.13 #4

Closed
wants to merge 1 commit into from

Conversation

Sabanic-P
Copy link
Contributor

@Sabanic-P Sabanic-P commented Oct 23, 2022

The version only had minor code changes.
A simple Test program runs without any problem.

Signed-off-by: Patrick Sabanic [email protected]

@Sabanic-P Sabanic-P changed the title Update version: 12.11 -> 1.2.12 Update version: 1.2.12 -> 1.2.13 Oct 23, 2022
@Sabanic-P Sabanic-P mentioned this pull request Oct 23, 2022
@razvand razvand added enhancement New feature or request kind/hackathon Hackathon challenge labels Oct 23, 2022
@razvand razvand linked an issue Oct 23, 2022 that may be closed by this pull request
@razvand razvand self-assigned this Oct 23, 2022
@razvand razvand self-requested a review October 23, 2022 14:45
Copy link

@razvand razvand left a comment

Choose a reason for hiding this comment

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

Hi, @Sabanic-P . Thanks for this. Please improve your commit message. Present tense: "Update library version to 1.2.13". Please also mention in the extended commit message the test used.

Follow this link for commit messages, also mentioned in the Submitting Changes page.

@razvand
Copy link

razvand commented Oct 23, 2022

@Sabanic-P , 20 points awarded. 20 points more for solving the comments for the PR to be merged.

Copy link

@razvand razvand left a comment

Choose a reason for hiding this comment

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

Thanks, @Sabanic-P

Reviewed-by: Razvan Deaconescu [email protected]
Approved-by: Razvan Deaconescu [email protected]

@razvand
Copy link

razvand commented Oct 23, 2022

@Sabanic-P , 20 more hackathon points added for the approval.

@NECJ
Copy link

NECJ commented Nov 16, 2022

Hi, can this please be merged because the 1.2.12 version is still causing issues when building.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/merged enhancement New feature or request kind/hackathon Hackathon challenge
Projects
Status: Done
Status: Done!
Development

Successfully merging this pull request may close these issues.

Update library to 1.2.13
4 participants