Skip to content

NOJIRA: process overhead, merge develop -> main (clean integration)#101

Merged
brendanobra merged 128 commits into
mainfrom
pr100-main-integration
Jul 15, 2026
Merged

NOJIRA: process overhead, merge develop -> main (clean integration)#101
brendanobra merged 128 commits into
mainfrom
pr100-main-integration

Conversation

@brendanobra

Copy link
Copy Markdown
Contributor

Conflict-resolved integration branch for develop -> main merge.

What changed:

  • Started from main and merged develop with explicit conflict resolution.
  • Resolved merge conflicts in workflow/license/version/runtime files.
  • Added a minimal gateway unit-test stabilization for async/no-server connect paths to remove flaky fixed-port assumptions.

Validation:

  • ./fmt.sh --fix
  • ./fmt.sh
  • ./test.sh (129/129 passing)

This PR is intended to supersede #100 with a clean, mergeable branch.

Copilot AI review requested due to automatic review settings July 14, 2026 19:54
@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/firebolt-cpp-transport/101/rdkcentral/firebolt-cpp-transport

  • Commit: 0b20234

Report detail: gist'

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 56 out of 58 changed files in this pull request and generated 2 comments.

Files not reviewed (1)
  • .github/release/package-lock.json: Generated file

Comment thread src/transport.cpp
Comment thread test/unit/transportTest.cpp
Copilot AI review requested due to automatic review settings July 14, 2026 20:07
@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/firebolt-cpp-transport/101/rdkcentral/firebolt-cpp-transport

  • Commit: 802d182

Report detail: gist'

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 56 out of 58 changed files in this pull request and generated 1 comment.

Files not reviewed (1)
  • .github/release/package-lock.json: Generated file

Comment thread src/transport.cpp Outdated
Copilot AI review requested due to automatic review settings July 14, 2026 20:20
@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/firebolt-cpp-transport/101/rdkcentral/firebolt-cpp-transport

  • Commit: 6e072c3

Report detail: gist'

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 56 out of 58 changed files in this pull request and generated 2 comments.

Files not reviewed (1)
  • .github/release/package-lock.json: Generated file

Comment thread .github/skills/openspec-coverage/openspec_coverage.py
Comment thread src/logger.cpp
Copilot AI review requested due to automatic review settings July 14, 2026 20:49
@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 1 file pending identification.

  • Protex Server Path: /home/blackduck/github/firebolt-cpp-transport/101/rdkcentral/firebolt-cpp-transport

  • Commit: 2138256

Report detail: gist'

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 56 out of 58 changed files in this pull request and generated 2 comments.

Files not reviewed (1)
  • .github/release/package-lock.json: Generated file

Comment thread src/logger.cpp
Comment thread src/transport.cpp
Copilot AI review requested due to automatic review settings July 14, 2026 21:10
@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

b'## Blackduck scan failure details

Summary: 0 violations, 0 files pending approval, 2 files pending identification.

  • Protex Server Path: /home/blackduck/github/firebolt-cpp-transport/101/rdkcentral/firebolt-cpp-transport

  • Commit: 12b456f

Report detail: gist'

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 56 out of 58 changed files in this pull request and generated no new comments.

Files not reviewed (1)
  • .github/release/package-lock.json: Generated file

@rdkcmf-jenkins

Copy link
Copy Markdown
Contributor

b'## WARNING: A Blackduck scan failure has been waived

A prior failure has been upvoted

  • Upvote reason: ok

  • Commit: 12b456f
    '

@brendanobra
brendanobra merged commit 16b8f37 into main Jul 15, 2026
12 of 13 checks passed
@brendanobra
brendanobra deleted the pr100-main-integration branch July 15, 2026 14:56
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 15, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants