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.

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

OSOE-353: Add duplicate SQL query detector in Lombiq.UITestingToolbox #247

Open
wants to merge 59 commits into
base: dev
Choose a base branch
from
Open

OSOE-353: Add duplicate SQL query detector in Lombiq.UITestingToolbox #247

wants to merge 59 commits into from

Conversation

dministro
Copy link
Member

@dministro dministro commented Oct 24, 2022 •
edited by jira bot

@dministro
Adding negative tests
714dd8c 
Adjusting tests
Updating submodule:
 - Lombiq.UITestingToolbox: merge from dev
@dministro
Merge branch 'dev' of https://github.com/Lombiq/Open-Source-Orchard-C…
cd9fc4e 
…ore-Extensions into issue/OSOE-353
@github-actions
Copy link

This pull request has merge conflicts. Please resolve those before requesting a review.

@dministro
Merge branch 'dev' of https://github.com/Lombiq/Open-Source-Orchard-C…
21429cc 
…ore-Extensions into issue/OSOE-353
@dministro
Updating submodules:
356def6 
 - Lombiq.UITestingToolbox: Implementing required exception constructors
@github-actions

This comment has been minimized.

Sign in to view
@dministro
Adjusting spelling configuration
71f517c 
Updating submodules:
 - Lombiq.HelpfulLibraries: fast-forward
 - Lombiq.OrchardCoreApiClient: fast-forward
 - Lombiq.HelpfulExtensions: fast-forward
 - Lombiq.Privacy: fast-forward
 - Lombiq.TrainingDemo: fast-forward
 - Lombiq.VueJs: fast-forward
 - Lombiq.Npm.Targets: fast-forward
 - Lombiq.UITestingToolbox: Adjusting tests Fixing typos
 - Lombiq.Analyzers.PowerShell: fast-forward
 - Lombiq.GitHub.Actions: Adjusting allow.txt
@github-actions

This comment has been minimized.

Sign in to view
@dministro
Updating submodules:
202de5f 
 - Lombiq.UITestingToolbox: Fixing typo
 - Lombiq.GitHub.Actions: Adjusting allow.txt
@github-actions
Copy link

This pull request has merge conflicts. Please resolve those before requesting a review.

@github-actions
Copy link

This pull request has merge conflicts. Please resolve those before requesting a review.

@github-actions
Copy link

This pull request has merge conflicts. Please resolve those before requesting a review.

@Piedone
Copy link
Member

Piedone commented Nov 2, 2022

On the next commit, UI testing will break if you don't merge from dev beforehand.

@dministro
Copy link
Member Author

On the next commit, UI testing will break if you don't merge from dev beforehand.

#258 , Thank you.

@dministro
Moving and renaming test classes
b6ee7ea
@dministro
Updating submodule:
911e7e4 
 - Lombiq.UITestingToolbox: Adding better exclusion and defaults
@github-actions

This comment has been minimized.

Sign in to view
@dministro
Updating submodule:
88422d3 
 - Lombiq.UITestingToolbox: Fix spelling
@dministro
Reverting changes
f396df0
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 5, 2023

This pull request has merge conflicts. Please resolve those before requesting a review.

@github-actions GitHub Actions
Copy link

This pull request has merge conflicts. Please resolve those before requesting a review.

@dministro
Updating submodule:
cccc2b6 
 - Lombiq.UITestingToolbox: Fixing analyzer violations
@dministro
Updating submodule:
1053c09 
 - Lombiq.UITestingToolbox: Fixing analyzer violations again
@github-actions GitHub Actions
Copy link

This pull request has merge conflicts. Please resolve those before requesting a review.

@github-actions GitHub Actions
Copy link

This pull request has merge conflicts. Please resolve those before requesting a review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Piedone Piedone Piedone approved these changes

Assignees
No one assigned
Labels
Merge Conflict requires-windows-build
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@dministro @Piedone @LombiqBot