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

moved flow helpers under otg config helpers , as flow is also configu… #3548

Merged
merged 1 commit into from
Oct 29, 2024

Conversation

Surajrawal-eqa
Copy link
Contributor

…ration in otg

@Surajrawal-eqa Surajrawal-eqa requested a review from a team as a code owner October 29, 2024 10:55
@OpenConfigBot
Copy link

Pull Request Functional Test Report for #3548 / 67bab8d

No tests identified for validation.

Help

@coveralls
Copy link

Pull Request Test Coverage Report for Build 11572396010

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 55.268%

Totals Coverage Status
Change from base Build 11569788103: 0.0%
Covered Lines: 1983
Relevant Lines: 3588

💛 - Coveralls

@Surajrawal-eqa Surajrawal-eqa merged commit 512cfb0 into openconfig:main Oct 29, 2024
13 checks passed
Copy link
Member

Choose a reason for hiding this comment

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

The file names stutter. Recommend naming like:L

otg_helpers/config/flow/

ANISH-GOTTAPU pushed a commit to open-traffic-generator/featureprofiles that referenced this pull request Nov 28, 2024
alshabib pushed a commit to alshabib/featureprofiles that referenced this pull request Jan 19, 2025
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.

5 participants