Skip to content

Namsonx/task/stabi branch #229

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 28 commits into from
Mar 11, 2024
Merged

Namsonx/task/stabi branch #229

merged 28 commits into from
Mar 11, 2024

Conversation

@namsonx
Copy link
Collaborator

@namsonx namsonx commented Mar 6, 2024

Hello Thomas,

I create this pull-request for JsonPreprocessor's code optimization plus some bugs fixing.

Thank you,
Son

HolQue and others added 15 commits February 5, 2024 16:07
@HolQue
Self test maintenance
590d12e
@HolQue
Merge branch 'develop' into HolQue/task/selftest_maintenance
8c60bc0
@namsonx
Fix bug 211 - Problems while accessing dictionary keys
6a25b1a
@namsonx
Optimize regular expression patterns
76d73d7
@namsonx
Bug fix 209 - Value change of dictionary key does not work
6c699e1
@HolQue
Merge remote-tracking branch 'origin/namsonx/task/stabi_branch' into …
f30e532 
…HolQue/task/selftest_maintenance
@namsonx
Ticket 210 - Improvement duplicated keys handling
926feca
@namsonx
Tiny change regarding to ticket 211
b39e602
@HolQue
Merge remote-tracking branch 'origin/namsonx/task/stabi_branch' into …
cf455bf 
…HolQue/task/selftest_maintenance
@HolQue
Test cases added (but commented out because they belong to current is…
aa467aa 
…sues)
@namsonx
Optimize JsonPreprocessor code
b8c705d
@test-fullautomation
Merge pull request #216 from test-fullautomation/HolQue/task/selftest…
d45e819 
…_maintenance

Self test maintenance
@HolQue
Resolved merge conflict caused by HolQue/task/documentation_maintenance
6981301 
(includes the documentation maintenance)
@test-fullautomation
Merge pull request #228 from test-fullautomation/HolQue/task/selftest…
53536ab 
…_maintenance

Resolved merge conflict caused by HolQue/task/documentation_maintenance
@namsonx
Optimize JsonPreprocessor code.
9666154
HolQue
HolQue reviewed Mar 6, 2024
View reviewed changes
Copy link
Collaborator

@HolQue HolQue left a comment

Choose a reason for hiding this comment

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

Yesterday I resolved a merge conflict caused by interface descriptions adapted by me. I also adapted the interface description of __nestedParamHandler. In this pull request my changes are gone and the previous version is back.

Accidental copy&paste issue?

@namsonx
Copy link
Collaborator Author

namsonx commented Mar 6, 2024

Yesterday I resolved a merge conflict caused by interface descriptions adapted by me. I also adapted the interface description of __nestedParamHandler. In this pull request my changes are gone and the previous version is back.

Accidental copy&paste issue?

Hello Holger,

This is the accident when I merged, I will check and update your descriptions

Thank you,
Son

@HolQue
Copy link
Collaborator

HolQue commented Mar 6, 2024

JsonPreprocessor still freezes. But now at another position within the test sequence.

I will try to find the exact position.

@namsonx
Copy link
Collaborator Author

namsonx commented Mar 6, 2024

Yesterday I resolved a merge conflict caused by interface descriptions adapted by me. I also adapted the interface description of __nestedParamHandler. In this pull request my changes are gone and the previous version is back.
Accidental copy&paste issue?

Hello Holger,

This is the accident when I merged, I will check and update your descriptions

Thank you, Son

Hello Holger,

I updated the descriptions back.

Thank you!

@test-fullautomation test-fullautomation added enhancement New feature or request 0.11.0 labels Mar 7, 2024
@test-fullautomation test-fullautomation added this to the 0.11.0 milestone Mar 7, 2024
@test-fullautomation test-fullautomation merged commit e882d7a into develop Mar 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@HolQue HolQue HolQue left review comments

@milanac030988 milanac030988 Awaiting requested review from milanac030988

@ngoan1608 ngoan1608 Awaiting requested review from ngoan1608

@huavanthong huavanthong Awaiting requested review from huavanthong

@trimai3001 trimai3001 Awaiting requested review from trimai3001

@test-fullautomation test-fullautomation Awaiting requested review from test-fullautomation

Assignees

No one assigned

Labels

0.11.0 enhancement New feature or request

Projects

RobotFramework AIO
Status: Done

Milestone

0.11.0

Development

Successfully merging this pull request may close these issues.

4 participants

@namsonx @HolQue @test-fullautomation