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 Version '1.12.2' into 'master' Branch #166

Merged
merged 7 commits into from
Feb 7, 2024
Merged

Release Version '1.12.2' into 'master' Branch #166

merged 7 commits into from
Feb 7, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Feb 6, 2024

🚀 Release '1.12.2' into 'master' Branch 🚀

This PR marks a pivotal moment in our deployment cycle, signaling that all changes on the Release branch are deemed ready for production. It represents the collective decision of our developers that the changes bundled in the Release are suitable to be released together.

What's Happening in This PR?

  • We are merging the 'release' branch into the 'master' branch.
  • This action is a critical step, transitioning us from the release phase to the production phase.

✅ Automatic Merging Upon CI Checks ✅

This PR will be automatically merged into the 'master' branch, if the following conditions are met:

  • All CI Checks pass
  • Code Review is approved by at least one developer

⚠️ Manual Merging Upon CI Checks and Code Review ⚠️

If the above conditions are not met, this PR will be manually merged into the 'master' branch, by a developer.

github-actions bot and others added 6 commits February 6, 2024 22:11
* ci(gitops): rename automatically created "boarding to train" branch, to 'boarding-auto'

* ci(gitops): fix syntax of merge-rt-in-release Workflow

* ci(gitops): fix merge-rt-in-release Workflow

* test: Regression and Backwards Compatibility

Add Automated Regression Test for 'Generator Docs Default Settings'. By Default Generator v2.x promises that upon invocation, the generated Project with be configured for Doc Builds with Sphinx and RTD Python3.8. The Test will fail if Defaults change. If Test is fixed, then this is evidence that if changes are released, they possibly introduce "breaking changes". Also, we add a new Test on the Dialog Prompts to verify that they prompt User for each and every one of the Cookiecutter Tempalte Variables. If this test fails, then the Dialog System and the Cookiecutter Tempalte Vars are out of Sync, which should be addressed, preventing a Bug from shipping Code with the 2 components being NOT synced. From this commit and on, we also stop include Code Coverage for Test Code ('tests' dir), but rather only for the Production Code, in 'src' dir.
Tests, for automatic compati and increase  Testsmeasure Code Coverage only for Code in 'src', and not 'tests', dir

test(dialog-system): verify prompts user for each and everyone of Cookiecutter Template Variables

test(docs-settings): verify Gen Project configured with Sphinx and RTD Py3.8, as advertised Defaults

refactor(isort): apply isort

refactor(black): apply black

* ci(gitops): simplify logic and debug

* ci(gitops): fix syntax of merge-rt-in-release workflow

* ci(gitops): track remote branches only if they do not exist locally

---------

Co-authored-by: konstantinos <[email protected]>
Co-authored-by: Konstantinos Lampridis <[email protected]>
Copy link

codecov bot commented Feb 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (df69397) 83.71% compared to head (eac2665) 88.42%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #166      +/-   ##
==========================================
+ Coverage   83.71%   88.42%   +4.71%     
==========================================
  Files          77       53      -24     
  Lines        1989      916    -1073     
==========================================
- Hits         1665      810     -855     
+ Misses        324      106     -218     
Flag Coverage Δ
unittests 88.42% <100.00%> (+4.71%) ⬆️

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.

boromir674
boromir674 previously approved these changes Feb 7, 2024
@boromir674 boromir674 self-assigned this Feb 7, 2024
@boromir674 boromir674 added test backwards-compatible Indicates, no Public API changes to be expected release Use when a Pull Request "carries" only release-related commits ie version bump, changelog update, .. labels Feb 7, 2024
@boromir674 boromir674 disabled auto-merge February 7, 2024 11:52
@boromir674 boromir674 merged commit 276f9e8 into master Feb 7, 2024
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backwards-compatible Indicates, no Public API changes to be expected business_logic docs release Use when a Pull Request "carries" only release-related commits ie version bump, changelog update, .. test
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant