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

release(1.6.0-RC6): merge release into main #211

Merged
merged 37 commits into from
Aug 16, 2023
Merged

Conversation

evegufy
Copy link
Contributor

@evegufy evegufy commented Aug 11, 2023

Description

Merge release branch for v1.6.0-RC6 into main

Why

to wrap up release process

Issue

N/A

Checklist

  • I have performed a self-review of my own code

nidhigarg-bmw and others added 30 commits July 26, 2023 15:50
* update change log for v1.6.0-RC5
* bump version for v1.6.0-RC5

---------

Co-authored-by: jjeroch <[email protected]>
Reviewed-By: Martin Rohrmeier <[email protected]>
nidhigarg-bmw and others added 7 commits August 10, 2023 03:00
Locales updated for app/service release process
why:
Placeholder have been set while the development process was ongoing, those placeholders getting now step-by-step replaced with more meaningful explanations (such as page descriptions)
Squashed commit of the following:

commit 70c0a71
Author: jjeroch <[email protected]>
Date:   Fri Aug 11 06:26:00 2023 +0200

    Update CHANGELOG.md

commit 5a96adb
Author: jjeroch <[email protected]>
Date:   Fri Aug 11 06:25:39 2023 +0200

    release/ChangeLog_1.6.0 RC-6
@sonarcloud
Copy link

sonarcloud bot commented Aug 11, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@evegufy
Copy link
Contributor Author

evegufy commented Aug 11, 2023

@oyo @nidhigarg-bmw @jjeroch could you please review this PR?
Due to the fact that the last release branches were squash merged into main and dev by mistake and I had to solve some conflicts, I want to make sure that everything in this PR is as expected.

@oyo Please don't merge this PR, I'll do that myself.

cc: @Phil91

Copy link
Contributor

@nidhigarg-bmw nidhigarg-bmw left a comment

Choose a reason for hiding this comment

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

lftm

@evegufy
Copy link
Contributor Author

evegufy commented Aug 16, 2023

@jjeroch I was just mentioning you at this pr for this 9f6de9b changes, I just checked, they should be fine.

So, I'm going ahead with the merge.

@evegufy evegufy merged commit bdf4a48 into main Aug 16, 2023
5 checks passed
@evegufy evegufy deleted the merge-v1.6.0-RC6-in-main branch August 16, 2023 12:17
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.

7 participants